On Thu, 4 Jan 2024 19:01:31 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> Justin Lu has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - cleanup comments
>>  - provide both SPI classes under build for relevant test files
>
> test/jdk/java/util/Locale/LocaleProvidersFormat.java line 30:
> 
>> 28:  * @library /test/lib
>> 29:  * @build LocaleProviders
>> 30:  *        providersrc.spi.src.tznp8013086
> 
> Although it is not needed in this test, I would list `tznp` to be built here, 
> as the provider's meta-info includes both classes for completeness. Applies 
> to `LocaleProvidersTimeZone` test too.

That makes sense, updated both the relevant test files as you suggested. Thank 
you for taking the time to review this.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17257#discussion_r1442166693

Reply via email to