RFR: 8276186: Require getAvailableLocales() methods to include Locale.ROOT [v2]
Naoto Sato
naoto at openjdk.java.net
Wed Nov 10 19:05:21 UTC 2021
On Wed, 10 Nov 2021 18:29:06 GMT, Pavel Rappo <prappo at openjdk.org> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Using @code tag for `Set`.
>
> src/java.base/share/classes/java/time/format/DecimalStyle.java line 118:
>
>> 116: * Lists all the locales that are supported.
>> 117: * <p>
>> 118: * At a minimum, the returned Set must contain a {@code Locale} instance equal to
>
> A nit, really. Consider applying either of these suggestions:
> Suggestion:
>
> * At a minimum, the returned {@code Set} must contain a {@code Locale} instance equal to
>
> Suggestion:
>
> * At a minimum, the returned set must contain a {@code Locale} instance equal to
Thanks. Adopted the former suggestion.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6258
More information about the core-libs-dev
mailing list