[13] RFR: 8224105: Cannot parse JapaneseDate string on some specified locales
Brent Christian
brent.christian at oracle.com
Tue May 21 20:36:41 UTC 2019
Thanks. Looks good to me.
-Brent
On 5/21/19 11:33 AM, naoto.sato at oracle.com wrote:
> Thanks, Brent. Modified the webrev accordingly:
>
> http://cr.openjdk.java.net/~naoto/8224105/webrev.01/
>
> Naoto
>
> On 5/21/19 11:03 AM, Brent Christian wrote:
>> Hi, Naoto. I have a couple comments.
>>
>> src/java.base/share/classes/sun/util/locale/provider/CalendarNameProviderImpl.java
>>
>>
>>
>> String.isEmpty() could be used in place of equals("").
>>
>>
>> test/jdk/java/time/test/java/time/chrono/TestEraDisplayName.java
>>
>>
>> Maybe give the new constants names in all-caps.
>>
>>
>> -Brent
>>
>> On 5/20/19 5:20 PM, naoto.sato at oracle.com wrote:
>>> Ping?
>>>
>>> Naoto
>>>
>>> On 5/17/19 3:43 PM, naoto.sato at oracle.com wrote:
>>>> Hi,
>>>>
>>>> Please review the fix to the following issue:
>>>>
>>>> https://bugs.openjdk.java.net/browse/JDK-8224105
>>>>
>>>> The proposed changeset is located at:
>>>>
>>>> https://cr.openjdk.java.net/~naoto/8224105/webrev.00/
>>>>
>>>> CLDR does not provide entire localized Japanese era names in locales
>>>> mentioned in the bug report. The parsing has been failing as they
>>>> have duplicate names, despite the duplicates are empty strings.
>>>>
>>>> Naoto
More information about the core-libs-dev
mailing list