On Thu, 21 Sep 2023 08:32:21 GMT, Alan Bateman <al...@openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   8316000: Move apiNotes to normative text; update @return descriptions
>
> src/java.base/share/classes/java/io/File.java line 1643:
> 
>> 1641:      *          change the access permissions of this abstract 
>> pathname.  If
>> 1642:      *          the underlying file system does not implement a read 
>> permission,
>> 1643:      *          then the operation will return the value of the {@code 
>> readable}
> 
> There's a switcharoo here. The method description speaks of the platform not 
> supporting file permissions. The return description changes terminology and 
> speaks of the file system not implementating permissions. I think we'll get 
> these consistent but overall the intent is okay.

Reconciled these in 8189784ace0a693a042f86505466806377684f80.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15673#discussion_r1334866057

Reply via email to