Re: RFR 8157936: Files.size(Path p) returns 0 if path is from JrtFileSystem with exploded build

2016-06-27 Thread Jim Laskey (Oracle)
+1 > On Jun 27, 2016, at 5:16 AM, Sundararajan Athijegannathan > 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 >>> wrote: >>> >>> Please review

Re: RFR 8157936: Files.size(Path p) returns 0 if path is from JrtFileSystem with exploded build

2016-06-27 Thread Sundararajan Athijegannathan
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 >> wrote: >> >> Please review http://cr.openjdk.java.net/~sundar/8157936/webrev.00/ for >> https://bugs.openjdk.java.net/brow

Re: RFR 8157936: Files.size(Path p) returns 0 if path is from JrtFileSystem with exploded build

2016-06-27 Thread Paul Sandoz
> On 27 Jun 2016, at 09:03, Sundararajan Athijegannathan > 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. Bu

RFR 8157936: Files.size(Path p) returns 0 if path is from JrtFileSystem with exploded build

2016-06-27 Thread Sundararajan Athijegannathan
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