On Wed, 3 Sep 2025 00:57:39 GMT, Roger Riggs <[email protected]> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Use Objects.requireNonNull()
>
> src/java.base/share/classes/java/text/DateFormatSymbols.java line 148:
>
>> 146: * if the resources for the specified locale cannot be
>> 147: * found or cannot be loaded.
>> 148: * @throws NullPointerException if {@code locale} is null
>
> Adding an explicit Objects.requireNull(locale, "locale") would clearly
> identify when the NPE is thrown.
> Though it might be earlier than without the check.
Right. Added the explicit null check. (and a test case)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27060#discussion_r2319328408
