Updated: http://cr.openjdk.java.net/~sundar/8159004/webrev.02/
-Sundar On 8/29/2016 5:52 PM, Alan Bateman wrote: > On 29/08/2016 13:00, Sundararajan Athijegannathan wrote: > >> Please review http://cr.openjdk.java.net/~sundar/8159004/webrev.01/ >> for https://bugs.openjdk.java.net/browse/JDK-8159004 >> >> Piggybacking couple of cleanups suggested (but missed) in this thread >> -> >> http://mail.openjdk.java.net/pipermail/jigsaw-dev/2016-August/009186.html > In storeFiles then you could move "Path bin = ..." to the enclosing > scope and avoid a bit of duplications, otherwise looks good. > > -Alan