<i18n dev> RFR: 8335668: NumberFormat integer only parsing should throw exception for edge case [v2]

Justin Lu jlu at openjdk.org
Tue Jul 9 20:29:48 UTC 2024


On Tue, 9 Jul 2024 19:41:29 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   reflect review
>
> test/jdk/java/text/Format/NumberFormat/LenientParseTest.java line 146:
> 
>> 144:     @EnabledIfSystemProperty(named = "user.language", matches = "en")
>> 145:     public void compactIntegerParseOnlyFractionOnlyTest() {
>> 146:         var fmt = NumberFormat.getIntegerInstance();
> 
> Should it get a compact number instance, instead of integer?

Thanks for catching, I would hope the compact test actually tests a compact format; fixed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20101#discussion_r1671197396


More information about the i18n-dev mailing list