RFR 8196535: Remove support for pre-Java 6 non-JVM-wide file locking

Roger Riggs Roger.Riggs at Oracle.com
Fri Feb 2 15:29:06 UTC 2018


Hi Brian,

Nice cleanup.

FileLockTable: line 80: typo "the a" -> "the"

FileLockTable:89:  the constructor should be private to avoid an end-run 
around the factory method.

While editing FileLockTable.java perhaps convert imports to single 
imports instead of .*... your call.

I'm assuming there were no tests for the old setting since there none in 
the review.

FileChannelImpl.java:1096: if the line doesn't need to be continued, 
maybe all on 1 line

Thanks, Roger

On 2/1/2018 6:40 PM, Brian Burkhalter wrote:
> https://bugs.openjdk.java.net/browse/JDK-8196535
> http://cr.openjdk.java.net/~bpb/8196535/webrev.00/
>
> This patch would remove support for the sun.nio.ch.disableSystemWideOverlappingFileLockCheck system property introduced in Java 6 to allow enabling pre-Java 6 behavior of checking for overlapping FileLocks opened on a FileChannel.
>
> Please note that this patch follows in sequence the patch proposed in [1] which still needs to pass review.
>
> Thanks,
>
> Brian
>
> [1] http://mail.openjdk.java.net/pipermail/nio-dev/2018-February/004667.html



More information about the nio-dev mailing list