On Tue, 5 Nov 2024 21:07:57 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> It looks like it's trying to be extra resilient by switching on >> `n.toLowerCase()` (i.e. case-insensitive). Is it OK to make it >> case-sensitive and use the constants? > > I thought about it too. Using constants and case-sensitive switch should be > enough. > > Also, I think the `switch` statement should have `continue` rather than > `break` for these logical fonts… unless the logical fonts are guaranteed to > be in the end of the array returned by > `GraphicsEnvironment.getAvailableFontFamilyNames()`. Done! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21716#discussion_r1830196351