+1
> On Jun 27, 2016, at 5:16 AM, Sundararajan Athijegannathan
> <[email protected]> wrote:
>
> 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.
>>
>
