RFR: 8344060: Remove doPrivileged calls from shared implementation code in the java.desktop module : part 1 [v4]

Alexey Ivanov aivanov at openjdk.org
Mon Nov 18 18:33:55 UTC 2024


On Mon, 18 Nov 2024 03:09:13 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

>> Since JEP 486 : Permanently Disable the Security Manager
>> [https://bugs.openjdk.org/browse/JDK-8338625] is now integrated, calls to java.security.AccessController.doPrivileged are obsolete and can be removed.
>> 
>> This PR takes care of some of the shared-platform files in the java.desktop module to have them removed.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove redundant assignment

Changes requested by aivanov (Reviewer).

src/java.desktop/share/classes/sun/awt/image/ImagingLib.java line 98:

> 96:         }
> 97: 
> 98:         useLib = Boolean.valueOf(success);

Suggestion:

        useLib = success;

Unnecessary wrapping into `Boolean`.

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

PR Review: https://git.openjdk.org/jdk/pull/22133#pullrequestreview-2443357726
PR Review Comment: https://git.openjdk.org/jdk/pull/22133#discussion_r1847043405


More information about the client-libs-dev mailing list