<i18n dev> RFR: 8349883: Locale.LanguageRange.parse("-") throws ArrayIndexOutOfBoundsException [v2]

Justin Lu jlu at openjdk.org
Wed Feb 12 22:06:35 UTC 2025


On Wed, 12 Feb 2025 21:40:48 GMT, Naoto Sato <naoto at openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Naoto's review - Include test in LanguageRangeTest instead. Additionally, bundle existing LRToString test into LanguageRangeTest as well
>
> test/jdk/java/util/Locale/IllformedRangeTest.java line 40:
> 
>> 38: import static org.junit.jupiter.api.Assertions.assertThrows;
>> 39: 
>> 40: public class IllformedRangeTest {
> 
> Instead of creating a new test file, you might want to add this test into `LanguageRangeTest.java`

Moved `IllformedRangeTest.java` as well as the existing `LRToString.java` test into `LanguageRangeTest.java`, which can now serve as a general class test. Also converted the test to JUnit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23596#discussion_r1953464752


More information about the i18n-dev mailing list