Rod-O commented on PR #64: URL: https://github.com/apache/xerces-c/pull/64#issuecomment-6546789909
A regression test for this change is on branch `XERCESC-2263-test` of https://github.com/zorba-processor/xerces-c (one commit on top of master 53c040181: https://github.com/zorba-processor/xerces-c/commit/6cf5230204ce0d0027e0e070cea07ac359cc1a59). It adds sample test `StdInParse4` (CMake and autotools), which validates xs:double and xs:float values against minInclusive, maxInclusive, minExclusive and maxExclusive: in-range values must be accepted and NaN must be rejected by all four facets. StdInParse reads the document from stdin, so the expected log contains no absolute paths. On master the test fails because the minInclusive and maxInclusive NaN values are accepted; with this PR applied the full ctest suite passes (81/81, Ubuntu 24.04, GCC 13, ICU transcoder). The commit can be cherry-picked onto this PR's branch as is. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
