[OpenJDK 2D-Dev] [PATCH] Refactor X11FontManager
Sergey.Bylokhov at oracle.com
Thu Feb 5 16:52:35 UTC 2015
Hello, Roman, Phil.
The fix looks fine except an absent documentation in new class and
InternalError + 80 chars per line.
But I have a question: do we really need this failback?
527 /* Using the X11 font path to locate font files is now a fallback
528 * useful only if fontconfig failed, or is incomplete. So we could
529 * remove this code completely and the consequences should be rare
530 * and non-fatal. If this happens, then the calling Java code can
531 * be modified to no longer require that the AWT lock (the X11GE)
532 * be initialised prior to calling this code.
On 04.02.2015 12:33, Roman Kennke wrote:
>>> I got no reply, so I'm sending it here, maybe it's a better fit? ;-)
>> font related discussions should go to 2d-dev.
>> The change looks OK. But test it as much as
>> you can. The previous re-factoring in JDK7
>> had a fairly long bug trail.
> Ok. I tested a couple of applications, including Java2Demo, Font2DTest
> and some more. They all work fine for me.
>> - Do submit a bug and mark it noreg-cleanup
>> - Do get a 2nd reviewer before committing.
> How do I do that? That seems to be the tricky part.. ;-)
>> - Do update the (C) in the new file to 2015 since the clock ticked
>> since you first sent the review.
> I only updated the (C) of the new file, can I leave the existing
> copyrights alone?
> Thanks & best regards,
>>> I am currently working on a port of AWT/Java2D to DirectFB (using
>>> Caciocavallo  as a porting layer).
>>> I hit one obstacle that is the X11FontManager. I want to use fontconfig
>>> to work with fonts, but can't use anything X11.
>>> I refactored X11FontManager and related classes and native code to avoid
>>> calling anything X11 specific. Basically I created a superclass of
>>> X11FontManager called FcFontManager that does all the fontconfig related
>>> stuff, and use that in my backend. The changes should not affect the X11
>>> Can you please review the changes here:
>>> I would like to push this into jdk9-client if you think it's ok.
>>> Best regards,
>>>  http://openjdk.java.net/projects/caciocavallo/
Best regards, Sergey.
More information about the 2d-dev