On Fri, 11 Sep 2026 11:12:46 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> Test used to fail on linux in CI in older releases but with upgrade in 
>> harfbuzz and linux, with current jdk, it is not failing now in several 
>> iterations in linux and other platforms (CI test job in JBS) so 
>> deproblemlisting 
>> 
>> 
>> 
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> 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 2:

> 1: /*
> 2:  * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.

date should be updated?

test/jdk/java/awt/Graphics2D/DrawString/DrawRotatedStringUsingRotatedFont.java 
line 95:

> 93:      */
> 94:     private static void compareImage(final BufferedImage bi1,
> 95:                                         final BufferedImage bi2) throws 
> IOException {

this will be longer than 80 chars?

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"?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32742#discussion_r4078208209
PR Review Comment: https://git.openjdk.org/jdk/pull/32742#discussion_r4078209141
PR Review Comment: https://git.openjdk.org/jdk/pull/32742#discussion_r4078223536

Reply via email to