RFR: 8003596 CheckLockLocationTest-Windows-fix

Jim Gish jim.gish at oracle.com
Mon Dec 3 15:01:43 UTC 2012


Thanks.  Could you please push the change.

Jim

On 12/01/2012 10:44 AM, Alan Bateman wrote:
> 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.

-- 
Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304
Oracle Java Platform Group | Core Libraries Team
35 Network Drive
Burlington, MA 01803
jim.gish at oracle.com




More information about the core-libs-dev mailing list