Looks OK.
I checked
https://docs.oracle.com/en/java/javase/12/intl/font-configuration-files.html
and I don't think it invalidates anything in there. We already removed
references to
the motif support which used to be there :
https://docs.oracle.com/javase/8/docs/technotes/guides/intl/fontconfig.html
-phil
On 5/18/19 3:10 PM, Sergey Bylokhov wrote:
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
