Re: RFR (JAXP) : 8003262 reverse translation required changes

2013-10-10 Thread huizhe wang
Hi Daniel, Lance, I've updated the webrev to include other resource bundles. The changes are the same. http://cr.openjdk.java.net/~joehw/jdk8/8003262/webrev/ Thanks, Joe On 10/9/2013 10:58 AM, Lance Andersen - Oracle wrote: +1 On Oct 9, 2013, at 1:35 PM, huizhe wang wrote: Hi, These

Re: RFR (JAXP) : 8003262 reverse translation required changes

2013-10-10 Thread Lance Andersen - Oracle
Hi Joe, I skimmed all of the changes and they look in line with the previous so +1 Best Lance On Oct 10, 2013, at 12:47 PM, huizhe wang wrote: Hi Daniel, Lance, I've updated the webrev to include other resource bundles. The changes are the same.

Re: RFR (JAXP) : 8003262 reverse translation required changes

2013-10-10 Thread huizhe wang
Thanks Lance for going through the changes! Joe On 10/10/2013 9:57 AM, Lance Andersen - Oracle wrote: Hi Joe, I skimmed all of the changes and they look in line with the previous so +1 Best Lance On Oct 10, 2013, at 12:47 PM, huizhe wang wrote: Hi Daniel, Lance, I've updated the webrev to

RFR (JAXP) : 8003262 reverse translation required changes

2013-10-09 Thread huizhe wang
Hi, These resource bundles in JAXP were refactored due to request by WPTG. Now that the WPTG tool has been approved to support the original format, I'm reverting the previous change back to the original format: http://cr.openjdk.java.net/~joehw/jdk8/8003262/webrev/ Thanks, Joe

Re: RFR (JAXP) : 8003262 reverse translation required changes

2013-10-09 Thread Daniel Fuchs
Hi Joe, I don't know what was the original format but these changes look reasonable. best regards, -- daniel (not a reviewer) On 10/9/13 7:35 PM, huizhe wang wrote: Hi, These resource bundles in JAXP were refactored due to request by WPTG. Now that the WPTG tool has been approved to support

Re: RFR (JAXP) : 8003262 reverse translation required changes

2013-10-09 Thread Lance Andersen - Oracle
+1 On Oct 9, 2013, at 1:35 PM, huizhe wang wrote: Hi, These resource bundles in JAXP were refactored due to request by WPTG. Now that the WPTG tool has been approved to support the original format, I'm reverting the previous change back to the original format: