> On Apr 29, 2016, at 5:38 AM, Alan Bateman <alan.bate...@oracle.com> wrote: > > The webrevs, all repos are here: > http://cr.openjdk.java.net/~alanb/8154956/1/
I reviewed all repos except hotspot. Looks good in general. Minor comments: Configuration.java Nit: line 361-364 there is an extra space sun/tools/jar/Main.java In the printModuleDescriptor method, creating a List from a stream is not necessary. 1668 md.requires().stream().sorted().collect(toList()).forEach( sortExports can be replaced as well. I have a patch to clean this up that I can push to jake: http://cr.openjdk.java.net/~mchung/jigsaw/webrevs/jar-jmod-cleanup/ Mandy