RFR: 8003596 CheckLockLocationTest-Windows-fix
Alan Bateman
Alan.Bateman at oracle.com
Sat Dec 1 15:44:51 UTC 2012
On 30/11/2012 23:19, Jim Gish wrote:
> Please review
> http://cr.openjdk.java.net/~jgish/Bug8003596-CheckLockLocationTest-Windows-fix/
> <http://cr.openjdk.java.net/%7Ejgish/Bug8003596-CheckLockLocationTest-Windows-fix/>
>
>
> Summary: fixes test when running on Windows so that test that requires
> setWritable is not run, because Windows does not support setWritable.
>
> Thanks,
> Jim
>
Looks okay to me although "if (!isWindows())" to "if (!ON_WINDOWS)"
might be neater. An alternative way to do this would be just to handle
check the return from setWritable rather than failing. It is possible to
change the deny adding entries to directories with the new file system
API but it's probably not worth using it here.
-Alan.
More information about the core-libs-dev
mailing list