Review request for JDK-8080266: Failed to create CharInfo due to ResourceBundle update for modules
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
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
Hi Many thanks for your review! Best Regards Frank -----Original Message----- From: Daniel Fuchs [mailto:daniel.fuchs@oracle.com] Sent: Wednesday, June 24, 2015 4:59 PM To: Frank Yuan; 'core-libs-dev'; 'Joe Wang' Cc: 'Lance Andersen'; 'jibing chen'; 'Gustavo Galimberti'; sandeep.konchady@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, 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
+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
So, would you like to push the code for me? Best Regards Frank -----Original Message----- From: huizhe wang [mailto:huizhe.wang@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@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
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@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@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
Alright, thanks a lot! :) Best Regards Frank -----Original Message----- From: Daniel Fuchs [mailto:daniel.fuchs@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.konchady@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.wang@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@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
participants (3)
-
Daniel Fuchs
-
Frank Yuan
-
huizhe wang