Hi Frank, The webrev is updated and looks good now.
Thanks for the great work! Joe On 11/13/2014 3:02 AM, Frank Yuan wrote:
Hi Joe and All I revised the code based on latest comments and put the webrev on http://cr.openjdk.java.net/~joehw/jdk9/test/Frank/8043090/webrev/ Best Regards Frank -----Original Message----- From: Frank Yuan [mailto:frank.y...@oracle.com] Sent: Wednesday, November 05, 2014 5:12 PM To: 'huizhe wang'; 'Core-Libs-Dev' Cc: 'jibing chen'; 'Gustavo Galimberti'; sandeep.konch...@oracle.com Subject: Review request for XML JAXP unit test colocation Hi, Joe and All Thanks for your previous reviews and comments. Per your comments: I added description for every test, got them running with and without security manager and fixed the known test issues. Could you help review the changes for JAXP unittest co-location again? After your review, I will push the tests into JDK repo: jaxp/test. bug: https://bugs.openjdk.java.net/browse/JDK-8043090 webrev: Thanks, Frank