On Fri, 15 Nov 2024 19:18:15 GMT, Phil Race <p...@openjdk.org> wrote:

> Update 21 desktop module shared implementation files to remove doPrivileged.
> All the usual tests still pass.

src/java.desktop/share/classes/sun/awt/datatransfer/DataTransferer.java line 
1013:

> 1011:     }
> 1012: 
> 1013:     private static ProtectionDomain 
> getUserProtectionDomain(Transferable contents) {

NB this particular source file has a lot more than doPrivileged that we should 
look at, but I stuck to doPrivileged, as is the brief, and submitted a bug to 
come back to this file later : https://bugs.openjdk.org/browse/JDK-8344256

src/java.desktop/share/classes/sun/font/CreatedFontTracker.java line 115:

> 113: 
> 114:         private static Thread t = null;
> 115:         @SuppressWarnings("removal")

This whole file may be obsolete now. But I stuck to the doPrivileged brief, and 
updated the existing tracker clean up bug to include looking at this file : 
https://bugs.openjdk.org/browse/JDK-8344146

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22161#discussion_r1844355388
PR Review Comment: https://git.openjdk.org/jdk/pull/22161#discussion_r1844357020

Reply via email to