On 4/16/12 2:38 PM, Joe Wang wrote: > I've added the testcase. The new webrev is here: > http://cr.openjdk.java.net/~joehw/jdk8/7160380/jdk/webrev/
Looks good. Thanks, Sean > > Also, Lance has reviewed the long list of the source changes. > > Thanks! > > -Joe > > On 4/16/2012 7:20 AM, Sean Mullan wrote: >> On 4/12/12 6:25 PM, Joe Wang wrote: >>> This is a follow-up update on Kelly's change that dropped the jaxp >>> source drop bundle process. With this update, the jaxp source in JDK8 >>> will be sync-ed with the current JAXP 1.4.5, and also JDK7u (the last >>> change was 7151484). Going forward, we'll follow the process to put any >>> changes into JDK8 before 7ux. >>> >>> webrev: http://cr.openjdk.java.net/~joehw/jdk8/7160380/webrev/ >>> >>> webrev for the change of package access: >>> http://cr.openjdk.java.net/~joehw/jdk8/7160380/jdk/webrev/ >> I only reviewed the package access webrev above. Could you also add the >> regression test that was added for 7u4: >> >> http://hg.openjdk.java.net/jdk7u/jdk7u4-dev/jdk/rev/e84d607e9b13 >> >> Looks good otherwise. >> >> Thanks, >> Sean
