On Mon, 21 Apr 2025 22:25:14 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   move future currencies set up under setUpTestingData()
>
> test/jdk/java/util/Currency/ValidateISO4217.java line 183:
> 
>> 181:         setUpPatchedClasses();
>> 182:         setUpTestingData();
>> 183:         setUpNotYetDefined();
> 
> It may be clearer to move this inside `setUpTestingData()`, and modify the 
> comment there

Right, makes more sense that way. Also updated the JBS issues as you requested.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24782#discussion_r2053093655

Reply via email to