On Wed, 13 Nov 2024 17:41:48 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 the windows-platform files in the java.desktop module 
> to have them removed.

src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/AnimationController.java
 line 28:

> 26: package com.sun.java.swing.plaf.windows;
> 27: 
> 28: import sun.security.action.GetBooleanAction;

Still needed ?

src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/AnimationController.java
 line 71:

> 69:     @SuppressWarnings("removal")
> 70:     private static final boolean VISTA_ANIMATION_DISABLED =
> 71:         
> Boolean.parseBoolean(System.getProperty("swing.disablevistaanimation"));

remove @SuppressWarnings("removal") ?
Boolean.getBoolean ?

src/java.desktop/windows/classes/sun/awt/windows/TranslucentWindowPainter.java 
line 69:

> 67:     // REMIND: we probably would want to remove this later
> 68:     private static final boolean forceOpt  =
> 69:         
> Boolean.parseBoolean(System.getProperty("sun.java2d.twp.forceopt", "false"));

Boolean.getBoolean ?

src/java.desktop/windows/classes/sun/awt/windows/TranslucentWindowPainter.java 
line 71:

> 69:         
> Boolean.parseBoolean(System.getProperty("sun.java2d.twp.forceopt", "false"));
> 70:     private static final boolean forceSW  =
> 71:         Boolean.parseBoolean(System.getProperty("sun.java2d.twp.forcesw", 
> "false"));

Boolean.getBoolean ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22083#discussion_r1841548349
PR Review Comment: https://git.openjdk.org/jdk/pull/22083#discussion_r1841547689
PR Review Comment: https://git.openjdk.org/jdk/pull/22083#discussion_r1841549225
PR Review Comment: https://git.openjdk.org/jdk/pull/22083#discussion_r1841549383

Reply via email to