Alright, thanks a lot! :) Best Regards Frank
-----Original Message----- From: Daniel Fuchs [mailto:daniel.fu...@oracle.com] Sent: Thursday, June 25, 2015 2:34 PM To: Frank Yuan; 'huizhe wang' Cc: 'core-libs-dev'; 'Lance Andersen'; 'jibing chen'; 'Gustavo Galimberti'; sandeep.konch...@oracle.com; 'Alexandre (Shura) Iline'; 'Alan Bateman' Subject: Re: Review request for JDK-8080266: Failed to create CharInfo due to ResourceBundle update for modules 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.w...@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.konch...@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 >>> >