<i18n dev> [15]: RFR: 8244245: localizedBy() should override localized values with default values

[Roger Riggs]

Looks good, thanks

On 5/7/20 12:06 PM, naoto.sato at oracle.com wrote:
> Hi Roger,
>
> Thank you for the review. Wrapped the long lines as suggested, along 
> with some typo fixes in the comments:
>
> https://cr.openjdk.java.net/~naoto/8244245/webrev.01/
>
> Naoto
>
> On 5/7/20 7:43 AM, Roger Riggs wrote:
>> Hi Naoto,
>>
>> Looks fine with a small source edit below.
>>
>> TestUnicodeExtension.java: Please wrap the excessively long lines; 
>> string concatination will put them together for the test.
>>
>> Thanks, Roger
>>
>>
>> On 5/6/20 4:44 PM, naoto.sato at oracle.com wrote:
>>> Hello,
>>>
>>> Please review the fix to the following issue:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8244245
>>>
>>> The CSR and proposed changeset are located at:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8244246
>>> https://cr.openjdk.java.net/~naoto/8244245/webrev.00/
>>>
>>> This stems from the closed issue 
>>> (https://bugs.openjdk.java.net/browse/JDK-8243162), and the 
>>> rationale for this fix is discussed there.
>>>
>>> Naoto
>>>
>>