[OpenJDK 2D-Dev] [13] Review Request: 8224171 The cleanup multi-font related code in the XFontPeer

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Sat May 18 22:10:31 UTC 2019


Hello.
Please review the fix for JDK 13.

Bug: https://bugs.openjdk.java.net/browse/JDK-8224171
Fix: http://cr.openjdk.java.net/~serb/8224171/webrev.00

This change is the part of my effort to clean up the native initialization code in awt/2d. Initially, I have dropped the native code inside initIds() which was unused(JDK-8223766[1]). Now I tried to check is the code inside initIDs() is used for some meaningful purpose.

I found that XFontPeer.initIds() uses XFontPeer.xfsname field which is always null, so in this fix, I have dropped the field itself, the initIds(), the methods which depend on this field, and methods which calls methods which depends on this field, etc. Next step will be clean up the native code for the Fonts itself, there is some unused code as well.

Note that this field was unused since 2009[2]

[1] https://bugs.openjdk.java.net/browse/JDK-8223766
[2] http://hg.openjdk.java.net/jdk/client/rev/a79da6a9d184

-- 
Best regards, Sergey.


More information about the 2d-dev mailing list