Thanks for the review. I'll include a space in test and push. -Sundar
On 6/27/2016 1:42 PM, Paul Sandoz wrote: >> 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. >
