open-source-parsers / jsoncpp

90%

LAST BUILD ON BRANCH master

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• 00.11.0
• 00.11.z
• 1.9.2
• 1.9.3
• 1.9.4
• 1.9.7
• Add-project-status-to-README-to-set-expectations
• BillyDonahue-avoid-isprint
• BillyDonahue-patch-1
• add-runtime-version
• allow-dropped-placeholders
• baylesj-patch-1
• baylesj-patch-6
• baylesj-patch-7
• baylesj-patch-8
• baylesj-patch-9
• ci/bump-node24-actions
• cmake-version
• coverage
• docs/remove-pkg-managers
• drop_cpptl_support
• feature-amalgamation
• fix-cpp20-build
• fix-cxx11-abi-break
• fix-czstring
• fix-fuzzer
• fix-github-actions-warnings
• fix-issue-1627
• fix/1427-subnormal-roundtrip
• fix/1500-trailing-comma-comment
• fix/1611-dense-array-index
• fix/1694-soversion-bump
• fix/cmake-dll-build-scope
• fix/locale-number-parsing
• fix/reader-dangling-ptr
• fix/reject-control-chars
• fix/string-view-abi
• fixup-project-version-updater
• fixup-tests
• fuzzer-fix1
• improve-formatting
• improve-qualifier-support
• inc
• issue1591
• json-dll-build
• jsoncpp_test
• master
• members
• patch-renu555
• pre-v2.y.z
• pypi
• refs/pull/1214/merge
• refs/pull/1339/merge
• refs/pull/1383/merge
• refs/pull/1409/merge
• refs/pull/1419/merge
• refs/pull/1424/merge
• refs/pull/1435/merge
• refs/pull/1448/merge
• refs/pull/1457/merge
• refs/pull/1462/merge
• refs/pull/1467/merge
• refs/pull/1478/merge
• refs/pull/1479/merge
• refs/pull/1480/merge
• refs/pull/1486/merge
• refs/pull/1491/merge
• refs/pull/1499/merge
• refs/pull/1516/merge
• refs/pull/1527/merge
• refs/pull/1528/merge
• refs/pull/1534/merge
• refs/pull/1567/merge
• refs/pull/1570/merge
• refs/pull/1572/merge
• refs/pull/1574/merge
• refs/pull/1575/merge
• refs/pull/1578/merge
• refs/pull/1579/merge
• refs/pull/1584/merge
• refs/pull/1593/merge
• refs/pull/1596/merge
• refs/pull/1597/merge
• refs/pull/1599/merge
• refs/pull/1600/merge
• refs/pull/1601/merge
• refs/pull/1602/merge
• refs/pull/1615/merge
• refs/pull/1619/merge
• refs/pull/1622/merge
• refs/pull/1625/merge
• refs/pull/1635/merge
• refs/pull/1637/merge
• refs/pull/1639/merge
• refs/pull/1648/merge
• refs/pull/1649/merge
• refs/pull/1654/merge
• refs/pull/1657/merge
• refs/pull/1658/merge
• refs/pull/1659/merge
• refs/pull/1660/merge
• refs/pull/1661/merge
• refs/pull/1662/merge
• refs/pull/1663/merge
• refs/pull/1665/merge
• refs/pull/1666/merge
• refs/pull/1667/merge
• refs/pull/1674/merge
• refs/pull/1677/merge
• refs/pull/1678/merge
• refs/pull/1679/merge
• refs/pull/1680/merge
• refs/pull/1683/merge
• refs/pull/1685/merge
• refs/pull/1686/merge
• refs/pull/1689/merge
• refs/pull/1691/merge
• refs/pull/1693/merge
• refs/pull/1695/merge
• refs/pull/1696/merge
• refs/pull/1697/merge
• refs/pull/1698/merge
• refs/pull/1699/merge
• refs/pull/1700/merge
• refs/pull/1701/merge
• refs/pull/1703/merge
• refs/pull/1706/merge
• refs/tags/1.9.6
• refs/tags/1.9.7
• refs/tags/1.9.8
• refs/tags/1.9.8-rc1
• revert-1662-fix/locale-number-parsing
• revert-soversion-change
• stack-depth
• v1.9.3
• version-header
• zizmor

Build # 28951244418

Build Type

push

github

Committed by web-flow

Commit Message

test: make ValueTest/objects float check robust against x87 excess precision (#1700)

ValueTest/objects stores the float literal 0.12345f into a Json::Value
(widened to the stored double) and then asserts equality against the
original 0.12345f. On 32-bit x86 targets where GCC defaults to x87 math
(-mfpmath=387, 80-bit excess precision) and the baseline lacks SSE2, the
round-trip yields the exact double 0.12345 rather than the float-widened
0.12345000356435776, so the exact double comparison fails even though the
library itself is correct.

Compare the stored value narrowed back to float via asFloat(): both the
expected literal and asFloat() collapse to the same float value on every
architecture, keeping the numeric round-trip check while making it
precision-robust.

Co-authored-by: Jordan Bayles <bayles.jordan@gmail.com>

Coverage Stats

2222 of 2640 branches covered (84.17%)

Branch coverage included in aggregate %.

2615 of 2740 relevant lines covered (95.44%)

23827.04 hits per line