Re: JDK 9 RFR of JDK-8075304: Remove duplicate test: FDTest

2015-03-31 Thread Amy Lu
Thank you Robert for your review ! May I get your help to sponsor this change? Thanks, Amy On 3/31/15 1:47 PM, Robert Field wrote: Looks good. -Robert On 03/30/15 20:44, Amy Lu wrote: Not yet got review feedback, I updated the webrev to get more cleanup included. Updated webrev: http://cr

Re: JDK 9 RFR of JDK-8075304: Remove duplicate test: FDTest

2015-03-30 Thread Robert Field
Looks good. -Robert On 03/30/15 20:44, Amy Lu wrote: Not yet got review feedback, I updated the webrev to get more cleanup included. Updated webrev: http://cr.openjdk.java.net/~amlu/8075304/webrev.01/ For your reference: For all tests proposed to be removed in this patch, here are the diffe

Re: JDK 9 RFR of JDK-8075304: Remove duplicate test: FDTest

2015-03-30 Thread Amy Lu
Not yet got review feedback, I updated the webrev to get more cleanup included. Updated webrev: http://cr.openjdk.java.net/~amlu/8075304/webrev.01/ For your reference: For all tests proposed to be removed in this patch, here are the differences between the one exist in "jdk" repo and "langtool

Re: JDK 9 RFR of JDK-8075304: Remove duplicate test: FDTest

2015-03-24 Thread Amy Lu
Kindly reminder ... On 3/17/15 3:05 PM, Amy Lu wrote: This RFR proposes to remove the duplicate FDTest from “jdk” repo. FDTest exists in both “jdk” and “langtools”: http://hg.openjdk.java.net/jdk9/dev/jdk/file/tip/test/jdk/lambda/FDTest.java http://hg.openjdk.java.net/jdk9/dev/langtools/file/

JDK 9 RFR of JDK-8075304: Remove duplicate test: FDTest

2015-03-17 Thread Amy Lu
This RFR proposes to remove the duplicate FDTest from “jdk” repo. FDTest exists in both “jdk” and “langtools”: http://hg.openjdk.java.net/jdk9/dev/jdk/file/tip/test/jdk/lambda/FDTest.java http://hg.openjdk.java.net/jdk9/dev/langtools/file/tip/test/tools/javac/lambdaShapes/org/openjdk/tests/javac/