RFR: 8269593: Different fontname on macos [v2]
Phil Race
prr at openjdk.java.net
Thu Jul 1 22:32:04 UTC 2021
On Tue, 29 Jun 2021 14:20:30 GMT, Johan Vos <jvos at openjdk.org> wrote:
>> Make sure the returned fullName of a created font matches the requested name. Since the name is used as a key/identifier in some cases, some internal code relies on this.
>> Added a test to check the case of "System Font Regular" on MacOS, which fails before and succeeds after the patch.
>
> Johan Vos has updated the pull request incrementally with one additional commit since the last revision:
>
> rename test
modules/javafx.graphics/src/main/java/com/sun/javafx/font/PrismFontFile.java line 612:
> 610: if (name != null) {
> 611: fullName = name;
> 612: }
So I suppose this is predicated on the assumption that the name being passed in is retrieved from the platform for *all* platforms (not just Mac) so we are sure it is a valid name ?
I'd be interested to see what the difference is in the full names we then display on each platform and whether locale matters .. the name got from the platform might be a locale name and that may not have been the intention here .. and is it possible "name" has already had toLowerCase() called on it ?
-------------
PR: https://git.openjdk.java.net/jfx/pull/553
More information about the openjfx-dev
mailing list