<i18n dev> 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 i18n-dev mailing list