On Fri, 30 May 2025 16:49:46 GMT, Harshitha Onkar <hon...@openjdk.org> wrote:

>> @prrace This is ready for review again. I've updated the code to combine 
>> nicely with the recent whitespace fixes. The `test/jdk/java/awt/font`, 
>> `test/jdk/java/awt/print` and `test/jdk/java/awt/Graphics2D/DrawString` 
>> tests all pass for me locally on Windows, Linux and macOS.
>> 
>> Once these changes are integrated it should be relatively simple to address 
>> JDK-8356803 ("Test TextLayout/TestControls fails on windows & linux: line 
>> and paragraph separator show non-zero advance") and JDK-8356812 ("Create an 
>> automated version of TextLayout/TestControls").
>
> @gredler In the process of reviewing this PR.
> 
> Just a headsup, JDK 25 RDP1 is next week 
> (https://openjdk.org/projects/jdk/25/) in case you are planning to get the 
> following fixes into 25:
> [JDK-8356803](https://bugs.openjdk.org/browse/JDK-8356803) - Test 
> TextLayout/TestControls fails on windows & linux: line and paragraph 
> separator show non-zero advance
> [JDK-8356812](https://bugs.openjdk.org/browse/JDK-8356812) - Create an 
> automated version of TextLayout/TestControls

@honkar-jdk Thanks! It shouldn't take me more than a day to address those two 
issues once this PR has been integrated, so I think it will just depend on how 
long the reviews take. I'll keep a close eye out to keep things quick on my 
end, though.

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

PR Comment: https://git.openjdk.org/jdk/pull/24412#issuecomment-2922956785

Reply via email to