On Fri, 30 May 2025 16:49:46 GMT, Harshitha Onkar <[email protected]> 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
