stevedlawrence opened a new pull request, #1587: URL: https://github.com/apache/daffodil/pull/1587
ICU 77.1 and older had a bug where it would not correctly parse infinity or NaN if the text pattern had a decimal and it was required to match a decimal point (which is enabled with strict parse policy). To workaround this bug, we added logic to parse a number second time if the first failed, but with different settings that did not require the decimal point, which allows it to detect infinity/NaN. But this adds extra overhead and complexity. ICU 78.1 fixed this bug, so we can now remove this workaround and avoid the overhead related to a second parse. This also updates the test that was added to detect if this bug was fixed, which now detects if ICU has a regression and breaks the fix. DAFFODIL-2985 -- 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]
