Review request for JDK-8080266: Failed to create CharInfo due to ResourceBundle update for modules

[Daniel Fuchs]

Hi Frank,

I could push it for you.

-- daniel

On 6/25/15 5:05 AM, Frank Yuan wrote:
> So, would you like to push the code for me?
>
> Best Regards
> Frank
>
> -----Original Message-----
> From: huizhe wang [mailto:huizhe.wang at oracle.com]
> Sent: Thursday, June 25, 2015 12:57 AM
> To: Daniel Fuchs
> Cc: Frank Yuan; 'core-libs-dev'; 'Lance Andersen'; 'jibing chen'; 'Gustavo
> Galimberti'; sandeep.konchady at oracle.com; 'Alexandre (Shura) Iline'; 'Alan
> Bateman'
> Subject: Re: Review request for JDK-8080266: Failed to create CharInfo due
> to ResourceBundle update for modules
>
> +1.
>
> -Joe
>
> On 6/24/2015 1:58 AM, Daniel Fuchs wrote:
>> Hi Frank,
>>
>> The proposed changes look good to me.
>>
>> best regards,
>>
>> -- daniel
>>
>> On 24/06/15 09:58, Frank Yuan wrote:
>>> Hi,
>>>
>>> Would you like to have a review for bug
>>> https://bugs.openjdk.java.net/browse/JDK-8080266?
>>>
>>> This bug is caused by jigsaw change, the context class loader can't
> load
>>> internal resource which is in a named module any more.
>>>
>>> To fix it, LSSerializerImpl shall invoke
>>> ResourceBundle.getBundle(resourceName) instead of
>>> ResourceBundle.getBundle(resourceName, locale, classloader) to create
>>> CharInfo instance, that will getBundle with the module of the
>>> caller(here it's java.xml module). This patch also forces to use the
>>> internal XMLEntities.properties because the default xml character
> entity
>>> reference should always be applied.
>>>
>>> The webrev is at:
>>> http://cr.openjdk.java.net/~fyuan/8080266/webrev.00/
>>>
>>> Any comment will be appreciated.
>>>
>>> Thanks,
>>>
>>> Frank
>>>
>