rework with mir - Githubissues

dlang-community / D-YAML

YAML parser and emitter for the D programming language

https://dlang-community.github.io/D-YAML/

Boost Software License 1.0

118 stars 38 forks source link

rework with mir #271

Closed 9il closed 3 years ago

9il commented 3 years ago

replace real with double to match other implementations: excess precision is bad when one need precise serialisation, fixes #270
use mir to/from string for floating point numbers, fixes #249
replace SysTime with Timestamp, arbitrary timestamp precision is supported now. For example, one can de/serialise Date without additional time. master can serialise only full precision timestamps.
added mir engine de/serialisation support for Node

Herringway commented 3 years ago

I'd rather not introduce new dependencies to DYAML.

9il commented 3 years ago

We need this changes or a fork. I expect that the fork will dominate after a while because of Symmetry.

Herringway commented 3 years ago

I cannot accept a dependency as large as mir-algorithm. This is not something I can maintain alone. There are also a large number of seemingly-irrelevant changes, such as the inclusion of an output.yaml, a submodule that appears to be unused, and private symbols being made public.

Threatening to fork will not address any of these issues. Competition is welcome.

9il commented 3 years ago

I hope you will change you mind. You wouldn't be alone to maintain it I think.