On Tue, 22 Oct 2024 20:20:16 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Sean Mullan has updated the pull request with a new target base due to a 
>> merge or a rebase. The pull request now contains 97 commits:
>> 
>>  - Merge remote-tracking branch 'jdk-sandbox/jep486' into JDK-8338411
>>  - Change apiNote to deprecated annotation on checkAccess methods. Change 
>> method dedescription to "Does nothing".
>>  - Sanitize the class descriptions of DelegationPermission and 
>> ServicePermission
>>    by removing text that refers to granting permissions, but avoid changes 
>> that
>>    affect the API specification, such as the description and format of input
>>    parameters.
>>  - Restored methods in RMIConnection to throw SecurityExceptions again but
>>    with adjusted text that avoids the word "permission".
>>  - Add text to class description of MBeanServer stating that implementations
>>    may throw SecurityException if authorization doesn't allow access to 
>> resource.
>>  - Restore text about needing permissions from the desktop environment in the
>>    getPixelColor and createScreenCapture methods.
>>  - Add api note to getClassContext to use StackWalker instead and
>>    add DROP_METHOD_INFO option to StackWalker.
>>  - Change checkAccess() methods to be no-ops, rather than throwing
>>    SecurityException.
>>  - Merge
>>  - Merge
>>  - ... and 87 more: https://git.openjdk.org/jdk/compare/f50bd0d9...f89d9d09
>
> src/java.base/share/classes/java/io/FilePermission.java line 71:
> 
>> 69:  *
>> 70:  * @apiNote
>> 71:  * This permission cannot be used for controlling access to resources 
>> anymore
> 
> The word "anymore" is unnecessary.

Fixed in 
https://github.com/openjdk/jdk/pull/21498/commits/44b552adb68d9aae1e72ed7bf20feb81552014c8

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21498#discussion_r1819108633

Reply via email to