RFR: JDK-8169505 - Update changes by JDK-8159393 to reflect CCC review

2016-11-15 Thread Jim Laskey (Oracle)
http://cr.openjdk.java.net/~jlaskey/8169505/webrev/index.html https://bugs.openjdk.java.net/browse/JDK-8169505

Re: [9] RFR: 8167240: Write new tests to cover functionality of existing 'jimage' options

2016-11-15 Thread Jim Laskey (Oracle)
+1 Really nice, thank you. > On Nov 15, 2016, at 11:16 AM, Denis Kononenko > wrote: > > > Hi, > > Please do re-review for these changes. > > 1) tests for list --include were rewritten accordingly to > https://bugs.openjdk.java.net/browse/JDK-8167384; > 2)

Re: [9] RFR: 8167240: Write new tests to cover functionality of existing 'jimage' options

2016-11-15 Thread Denis Kononenko
Hi, Please do re-review for these changes. 1) tests for list --include were rewritten accordingly to https://bugs.openjdk.java.net/browse/JDK-8167384; 2) removed tests for '@filename', see https://bugs.openjdk.java.net/browse/JDK-8169720; 3) two new CRs were submitted:

Re: RFR: JDK-8169720 - jimage help message for --include option should be corrected

2016-11-15 Thread Sundararajan Athijegannathan
+1 On 11/15/2016 8:09 PM, Jim Laskey (Oracle) wrote: > http://cr.openjdk.java.net/~jlaskey/8169720/webrev/index.html > > https://bugs.openjdk.java.net/browse/JDK-8169720 > >

Re: RFR: JDK-8169720 - jimage help message for --include option should be corrected

2016-11-15 Thread Alan Bateman
On 15/11/2016 14:39, Jim Laskey (Oracle) wrote: http://cr.openjdk.java.net/~jlaskey/8169720/webrev/index.html https://bugs.openjdk.java.net/browse/JDK-8169720 Looks fine.

RFR: JDK-8169720 - jimage help message for --include option should be corrected

2016-11-15 Thread Jim Laskey (Oracle)
http://cr.openjdk.java.net/~jlaskey/8169720/webrev/index.html https://bugs.openjdk.java.net/browse/JDK-8169720

Re: Compiling Java 9

2016-11-15 Thread Stephan Herrmann
Hi Alex, On 11/15/2016 12:20 AM, Alex Buckley wrote: You have a lot of questions here, mainly not about the OpenJDK implementation of JSR 379 but rather about JSR 379 itself. I'm working on JLS text that will be presented alongside lang-vm.html in the next few weeks, but here are some quick