On Wed, 23 Sep 2026 01:33:03 GMT, Sergey Bylokhov <[email protected]> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Use Dialog font
>
> test/jdk/java/awt/Graphics2D/DrawString/DrawRotatedStringUsingRotatedFont.java
>  line 143:
> 
>> 141:         bg.transform(gtx);
>> 142:         bg.setColor(Color.BLACK);
>> 143:         bg.setFont(font.deriveFont(20.0f).deriveFont(ftx));
> 
> please double check that initial bug is reproduced with "Font(Font.DIALOG, 
> Font.PLAIN, 20)". Why it is actually needed to be changed, and not covered 
> the default "system one"?

I have already done that..see this comment 
https://github.com/openjdk/jdk/pull/32742#issuecomment-5633538857
It needed to be changed since system default font can change from one release 
to another, mainly for linux, I presume, where sometimes  test expectation 
isn't met with that font, so we need a font which is stable across platforms.
Did you remember seeing any issues with different fonts present in the system 
when you were fixing the original issue?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32742#discussion_r4080601546

Reply via email to