Re: [10] RFR of JDK-8184329: Refactor java/lang/ClassLoader shell tests to java

2017-09-21 Thread mandy chung
On 9/21/17 9:31 PM, Amy Lu wrote: On 9/22/17 2:50 AM, mandy chung wrote: On 9/20/17 10:52 PM, Amy Lu wrote: test/jdk/java/lang/ClassLoader/getdotresource.sh Please review this patch to refactor the shell test to java. bug: https://bugs.openjdk.java.net/browse/JDK-8184329 webrev: http://cr

Re: [10] RFR of JDK-8184329: Refactor java/lang/ClassLoader shell tests to java

2017-09-21 Thread Amy Lu
On 9/22/17 2:50 AM, mandy chung wrote: On 9/20/17 10:52 PM, Amy Lu wrote: test/jdk/java/lang/ClassLoader/getdotresource.sh Please review this patch to refactor the shell test to java. bug: https://bugs.openjdk.java.net/browse/JDK-8184329 webrev: http://cr.openjdk.java.net/~amlu/8184329/webre

Re: [10] RFR of JDK-8184329: Refactor java/lang/ClassLoader shell tests to java

2017-09-21 Thread Amy Lu
On 9/22/17 2:50 AM, mandy chung wrote: On 9/20/17 10:52 PM, Amy Lu wrote: test/jdk/java/lang/ClassLoader/getdotresource.sh Please review this patch to refactor the shell test to java. bug: https://bugs.openjdk.java.net/browse/JDK-8184329 webrev: http://cr.openjdk.java.net/~amlu/8184329/webre

Re: [10] RFR of JDK-8184329: Refactor java/lang/ClassLoader shell tests to java

2017-09-21 Thread mandy chung
On 9/20/17 10:52 PM, Amy Lu wrote: test/jdk/java/lang/ClassLoader/getdotresource.sh Please review this patch to refactor the shell test to java. bug: https://bugs.openjdk.java.net/browse/JDK-8184329 webrev: http://cr.openjdk.java.net/~amlu/8184329/webrev.00/ I think you can replace line 64-6