RFR: 8246592: Simplify checking of boolean file attributes

Roger Riggs Roger.Riggs at oracle.com
Thu Jun 4 13:22:49 UTC 2020


Hi Claes,

Not a review...

You'll need a CSR for the API change.

FileSystem.java: You'll need proper javadoc for the new 
getBooleanAttribute method.

6/11 is Rampdown Phase One  (its a bit late for new APIs).

Regards, Roger


On 6/4/20 8:58 AM, Claes Redestad wrote:
> Hi,
>
> this patch proposes adding a FileSystem.getBooleanAttribute convenience
> method, which simplifies code at use sites. This also enables
> improvements when not requesting an attribute that requires extra work,
> such as getting the hidden bit on unix derivatives.
>
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8246592
> Webrev: http://cr.openjdk.java.net/~redestad/8246592/open.00/
>
> Testing: tier1-2
>
> Micro results on Linux:
>
> Benchmark                   Mode  Cnt  Score   Error  Units
> FileOpen.booleanAttributes  avgt   10  1.868 ± 0.002  us/op
>
> Benchmark                   Mode  Cnt  Score   Error  Units
> FileOpen.booleanAttributes  avgt   10  0.938 ± 0.008  us/op
>
> Thanks!
>
> /Claes



More information about the core-libs-dev mailing list