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