On Mon, 18 Nov 2024 03:09:13 GMT, Prasanta Sadhukhan <psadhuk...@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
