Thanks Joe answering this. Updated webrev by changing jaxp_test to jaxp_all, Also correct “Jaxp" as “JAXP”.
http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.01/ <http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.01/> http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.02/ <http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.02/> Tristan > On Dec 12, 2014, at 9:45 AM, huizhe wang <huizhe.w...@oracle.com> wrote: > > > On 12/12/2014 2:45 AM, Alan Bateman wrote: >> On 11/12/2014 21:10, Tristan Yan wrote: >>> Thanks Alan and Joe >>> I added jaxp testset, now jaxp_tests is under jaxp testset only. Also I >>> have added jaxp testset to pit. >>> >>> http://cr.openjdk.java.net/~tyan/JDK-8065673/webrev.01/ >>> <http://cr.openjdk.java.net/%7Etyan/JDK-8065673/webrev.01/> >>> http://cr.openjdk.java.net/~tyan/JDK-8065673/jaxp/webrev.00/ >>> <http://cr.openjdk.java.net/%7Etyan/JDK-8065673/jaxp/webrev.00/> >>> >>> Thanks >>> >> I think this looks much better. A suggestion for the test group name is >> "jaxp_all" rather than "jaxp_test". A minor comment on jprt.properties is >> that the comments should probably use "JAXP" rather than "Jaxp". >> >> A side question: Are there plans to move the JAXP tests from the jdk/test >> directory to the jaxp repo so that they can be with their friends? I think >> that would make this new testset more useful. > > Yes, it's in our plan to move jaxp tests from jdk/test to jaxp/test. > > -Joe > >> >> -Alan. >> >
