[OpenJDK 2D-Dev]  Review Request: 8224171 The cleanup multi-font related code in the XFontPeer
philip.race at oracle.com
Fri Jun 7 16:24:29 UTC 2019
and I don't think it invalidates anything in there. We already removed
the motif support which used to be there :
On 5/18/19 3:10 PM, Sergey Bylokhov wrote:
> 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). 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
>  https://bugs.openjdk.java.net/browse/JDK-8223766
>  http://hg.openjdk.java.net/jdk/client/rev/a79da6a9d184
More information about the 2d-dev