RFR: 7038105 - File.isHidden() should return true for pagefile.sys and hiberfil.sys

Alan Bateman Alan.Bateman at oracle.com
Fri May 24 17:06:01 UTC 2013


On 24/05/2013 17:50, Rob McKenna wrote:
> http://cr.openjdk.java.net/~robm/7038105/webrev.03/
>
>     -Rob
Good, it's right now.

Just a few comment comments:

- I'd suggest changing the comment on the function to something like 
"Read and return the attributes of the final target" as "special cases" 
is a bit confusing.

- The comment on line 230 says "pagefile.sys is a special case" but this 
covers all sharing issues now so I suggest removing it.

- For L230/231 then it might be clearer as:

    } else if (GetLastError() == ERROR_SHARING_VIOLATION) {

but that just a preference thing, what you have is okay too.

-Alan.



More information about the core-libs-dev mailing list