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