nlohmann / json / 38783114323

98%

Build Type

push

github

Committed by web-flow

Commit Message

Fix AppVeyor failures from the Zmij double writer (#5843)

MSVC 2015 warns C4554 on the shifts with casts in the fractional part
of zmij::to_shortest despite the parentheses; with /WX every target
fails to build. Name the shift amounts as constants instead.

The shortest-digits test built the candidates with one digit fewer via
a stream with std::setprecision(digits.size() - 2); the standard library
of MSVC 2017 ignores a precision of 0, so for two-digit values the
"shorter" candidates were the value itself. Truncate the digits
instead: the value lies within half a unit of the last digit, so the
truncated digits and their successor are the closest candidates.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>

Coverage Stats

3506 of 4043 branches covered (86.72%)

3 of 3 new or added lines in 1 file covered. (100.0%)

125 existing lines in 9 files now uncovered.

9521 of 9699 relevant lines covered (98.16%)

7155600.44 hits per line