On Tue, 6 Dec 2022 22:26:38 GMT, Phil Race <p...@openjdk.org> wrote: >> Olga Mikhaltsova has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Deleted trailing whitespaces, added new line at the end > > test/jdk/java/awt/font/LineBreakMeasurer/LineBreakWithTracking.java line 50: > >> 48: >> 49: private float textTracking = 0.0f; >> 50: private static String fontName = "Bitstream Cyberbit"; > > Did you previously use this font ? I must have over-looked that. > Do not use fonts that aren't available on the OS .. no SQE person will ever > install - or even know a test uses it - and JDK will just use Dialog instead > and people will get confused over why their experience is different than > yours. > And of course each OS (desktop) will have different fonts. Overall best to > just pick a logical font. If you want to know if the font supports all your > code points you can iterate over your text using Font.canDisplay(int > codePoint) > > I'll submit a test job using your automated test.
I noticed the usage of this font "Bitstream Cyberbit" in TestJustification.java that was added by you. I used the test string from there because it includes Latin, Arabic, CJK and Hebrew. "Bitstream Cyberbit" has all nessesary symbols used in the test string. At the moment I have no idea what font I can use instead. ------------- PR: https://git.openjdk.org/jdk/pull/10289
