Re: RFR(JDK 12/java.xml) 8213117: adoptNode corrupts attribute values

2018-11-28 Thread Joe Wang
Thanks Lance! On 11/28/18, 9:41 AM, Lance Andersen wrote: Hi Joe Looks good, Best Lance On Nov 27, 2018, at 2:58 PM, Joe Wang > wrote: Hi, Please review a fix for the adoptNode impl. Before a node from a deferred DOM can be adopted, it needs to be fully expan

Re: RFR(JDK 12/java.xml) 8213117: adoptNode corrupts attribute values

2018-11-28 Thread Lance Andersen
Hi Joe Looks good, Best Lance > On Nov 27, 2018, at 2:58 PM, Joe Wang wrote: > > Hi, > > Please review a fix for the adoptNode impl. Before a node from a deferred DOM > can be adopted, it needs to be fully expanded. > > JBS: https://bugs.openjdk.java.net/browse/JDK-8213117 > webrev: http://c

RFR(JDK 12/java.xml) 8213117: adoptNode corrupts attribute values

2018-11-27 Thread Joe Wang
Hi, Please review a fix for the adoptNode impl. Before a node from a deferred DOM can be adopted, it needs to be fully expanded. JBS: https://bugs.openjdk.java.net/browse/JDK-8213117 webrev: http://cr.openjdk.java.net/~joehw/jdk12/8213117/webrev/ Thanks, Joe