RFR: 8322166: Files.isReadable/isWritable/isExecutable expensive when file does not exist [v4]

Tue Dec 19 07:32:38 UTC 2023

On Mon, 18 Dec 2023 22:35:57 GMT, Chen Liang <liach at openjdk.org> wrote:

>>> why do we use redundant `? true : false` for a boolean value?
>> 
>> @liach What meaning do you intend by "redundant" here?
>
> I mean we can just use `access(file, R_OK) == 0`

One of the iterations did change the "exists" methods, the three isXXXX methods were missed. It's not a correctness issue of course, just a cleanup.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17133#discussion_r1431017472