[OpenJDK 2D-Dev] [PATCH] FontManager refactoring, rev6
Phil Race
Phil.Race at Sun.COM
Wed Aug 5 23:17:37 UTC 2009
Roman,
Roman Kennke wrote:
> Hi there,
>
> 6th round of the FontManager refactoring.
Can I assume you changed nothing except what you call out here ?
In which case I'm fine except for one thing :
>> WPathGraphics.java
>>
>> I see you moved the definition of textLayoutIsCompatible in here.
>> Whilst its used only by this code, its also meaning that updating
>> that font internal code in the future will affect the printing code.
>> And the getDirectoryEntry() method isn't public so I'm not sure
>> how this will work! Ah, I just remembered you said you didn't test
>> on windows yet. Puzzlement over, this won't build. I'd prefer this
>> to go back to TrueTypeFont.
you've resolved this by making getDirectoryEntry() public whereas
I asked that you move textLayoutIsCompatible back to the font code
where I believe it belongs.
-phil.
More information about the 2d-dev
mailing list