> On 27 Jun 2016, at 09:03, Sundararajan Athijegannathan > <[email protected]> wrote: > > Please review http://cr.openjdk.java.net/~sundar/8157936/webrev.00/ for > https://bugs.openjdk.java.net/browse/JDK-8157936 > > Note: no regrestion test added as jrt-fs automated testing is not done > in exploded mode JDK. But, I'm adding a simple class file size test to > Basic.java so that a manual run of jrt-fs tests on exploded image can > verify the fix. >
Look good. 146 return isDirectory()? 0 : Files.size(path); Trivially a missing space between ‘)’ and ‘?’. Paul.
