RFR: 8273581: Change the mechanism by which JDK loads the platform-specific FontManager class [v3]

Alexander Scherbatiy alexsch at openjdk.java.net
Fri Sep 17 12:41:38 UTC 2021


On Fri, 17 Sep 2021 07:57:06 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Alexander Scherbatiy has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Move createFontManager() code to sun.font.PlatformFontInfo class
>
> src/java.desktop/share/classes/sun/font/FontManagerFactory.java line 54:
> 
>> 52:      * @return a valid FontManager instance for the current platform
>> 53:      */
>> 54:     @SuppressWarnings("removal")
> 
> Suppress is not needed anymore?

I removed the SuppressWarnings annotation and obsolete imports.

> test/jdk/sun/awt/font/CheckFontManagerSystemProperty.java line 35:
> 
>> 33: 
>> 34:     public static void main(String[] args) {
>> 35:         // force AWT library loading
> 
> Do we need the headful or headless libraries?

I added `-Djava.awt.headless=true` property to run the test in the headless mode.

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

PR: https://git.openjdk.java.net/jdk/pull/5517



More information about the client-libs-dev mailing list