<i18n dev> RFR: 8344061: Remove doPrivileged calls from shared implementation code in the java.desktop module : part 2 [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Sun Nov 17 11:13:46 UTC 2024


On Sun, 17 Nov 2024 06:07:07 GMT, Phil Race <prr at openjdk.org> wrote:

>> Update 21 desktop module shared implementation files to remove doPrivileged.
>> All the usual tests still pass.
>
> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8344061

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

> 129:                      */
> 130:                     t.setContextClassLoader(null);
> 131:                     Runtime.getRuntime().addShutdownHook(t);

addShutdownHook is removed? accidentally?

src/java.desktop/share/classes/sun/font/SunLayoutEngine.java line 168:

> 166:     static boolean useFFM = true;
> 167:     static {
> 168:         @SuppressWarnings("removal")

SuppressWarning required?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22161#discussion_r1845409403
PR Review Comment: https://git.openjdk.org/jdk/pull/22161#discussion_r1845410575


More information about the i18n-dev mailing list