RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-02-28 Thread David Li
Hi, This is an update from Xerces for a fixed encoding map entry in file EncodingMap.java. For details, please refer to: https://bugs.openjdk.java.net/browse/JDK-8035469 Webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8035469/webrev/ (I don't have a openjdk username yet, so Joe Wang uploaded

Re: RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-02-28 Thread Lance Andersen - Oracle
Looks fine. On Feb 28, 2014, at 5:11 PM, David Li wrote: > Hi, > > This is an update from Xerces for a fixed encoding map entry in file > EncodingMap.java. For details, please refer to: > https://bugs.openjdk.java.net/browse/JDK-8035469 > > Webrevs: http://cr.openjdk.java.net/~joehw/jdk9/803

Re: RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-02-28 Thread huizhe wang
Looks good. I'll sponsor the change. Thanks, Joe On 2/28/2014 2:16 PM, Lance Andersen - Oracle wrote: Looks fine. On Feb 28, 2014, at 5:11 PM, David Li wrote: Hi, This is an update from Xerces for a fixed encoding map entry in file EncodingMap.java. For details, please refer to: https://

Re: RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-03-01 Thread Alan Bateman
On 28/02/2014 22:11, David Li wrote: Hi, This is an update from Xerces for a fixed encoding map entry in file EncodingMap.java. For details, please refer to: https://bugs.openjdk.java.net/browse/JDK-8035469 Webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8035469/webrev/ (I don't have a open

Re: RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-03-01 Thread David Li
Joe probably knows more about this, but we did some preliminary investigation summarized below. One test that was considered was creating an XML file encoded in one of the formats and then seeing if the parser would process the file after our updates were added. This looked like it requires g

Re: RFR (JAXP): 8035469 : Xerces Update: EncodingMap does not recognise Java-style encodings Cp1141-Cp1149

2014-03-03 Thread huizhe wang
Hi Alan and all, Thanks for bringing up the testing issue. For the Xerces Update project, I'd like to share our process for adding or creating tests. The basis of the process is that, with understanding that the Xerces release have been out for over 3 years and patches were generally verified