<i18n dev> RFR: 8054214: JapaneseEra.getDisplayName doesn't return names if it's an additional era

Jonathan Bluett-Duncan jbluettduncan at gmail.com
Thu Dec 8 11:52:47 UTC 2016

Hi Masayoshi,

I'm not a reviewer, but there's something in JapaneseEra.java.cdiff.html
isn't terribly clear to me, so I wonder if you could clarify it for me.

In the method `getDisplayName(TextStyle style, Locale locale)`, it's not
clear to me why the call to `Objects.requireNonNull(locale, "locale");` is
within the if-statement rather than on the first line. Is this because
`Era.super::getDisplayName` already has a null check of its own?

Kind regards,

On 8 December 2016 at 08:55, Masayoshi Okutsu <masayoshi.okutsu at oracle.com>

> Hi,
> Please review the fix for JDK-8054214. It was necessary to override
> Era::getDisplayName to get era names from the specified property. This one
> fixes getAbbreviation() as well.
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8054214
> Webrev:
> http://cr.openjdk.java.net/~okutsu/9/8054214/webrev.00/
> Thanks,
> Masayoshi

More information about the i18n-dev mailing list