toml-rs / toml / 15742043357

81%

Build Type

push

github

Committed by web-flow

Commit Message

feat(toml): Add support for bignum, u64, u128, i128 (#958)

Regarding integers, the TOML spec says

> Arbitrary 64-bit signed integers (from −2^63 to 2^63−1) should be
accepted and handled losslessly. If an integer cannot be represented
losslessly, an error must be thrown.

Regarding floats:

> Floats should be implemented as IEEE 754 binary64 values.

toml-lang/toml#1058 is clarifying these to mean that larger of both is
allowed.

This PR adds parse support to `toml` for big num, `u64`, `u128`, and
`i128`. This only adds deserialization support for `u64`, `u128`, and
`i128`. This does not yet add any serialization support or `toml::Value`
support for any of these.

As this just defers what we were already going to to and the happy path
was what we were already doing, I didn't bother putting this behind a
feature flag like `serde_json`.

Run Details

69 of 105 new or added lines in 9 files covered. (65.71%)

6 existing lines in 1 file now uncovered.

6228 of 8987 relevant lines covered (69.3%)

6.89 hits per line