[loc-en-dev] Regression - patch

[Doug Felt]

Looks good to me.

On Thu, Sep 23, 2010 at 11:51 AM, Naoto Sato <naoto.sato at oracle.com> wrote:

> Looks good to me. Can someone who has signed SCA review it as well? Steven
> or Doug I guess? This is needed to push the change.
>
> BTW, the bug id is 6986612, which will show up soon in the bug data base.
>
> Naoto
>
>
> (9/22/10 1:24 PM), Yoshito Umaoka wrote:
>
>> I'm attaching the patch in this message.
>>
>> The change in BaseLocale is not required, but I want to make the
>> consistent behavior -
>>
>> - Locale throws NPE when any locale field argument in the constructor is
>> null.
>> - BaseLocale#getInstance accepts null argument for all fields.
>>
>>
>> Thanks,
>> Yoshito
>>
>> On 09/21/2010 02:53 PM, Naoto Sato wrote:
>>
>>> You can just post the diff here (I guess it's small), and I'll review
>>> it and push it when it's ready.
>>>
>>> Naoto
>>>
>>> (9/21/10 11:54 AM), Yoshito Umaoka wrote:
>>>
>>>> I see. How can we provide a fix?
>>>>
>>>> On Tue, Sep 21, 2010 at 2:46 PM, Naoto Sato<naoto.sato at oracle.com>
>>>> wrote:
>>>>
>>>>> Looks like one of the test teams found a regression. The constructor
>>>>> Locale(lang, ctry, null) does not throw an NPE anymore.
>>>>>
>>>>> Naoto
>>>>>
>>>>>
>>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/locale-enhancement-dev/attachments/20100923/14816d47/attachment.html