RFR: 8312555: Hieroglyphs aren't stretched by AffineTransform.scale(2, 1)
Sergey Bylokhov
serb at openjdk.org
Thu Aug 17 21:50:28 UTC 2023
On Thu, 17 Aug 2023 21:42:04 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> test/jdk/java/awt/font/FontScaling/StretchedFontTest.java line 107:
>>
>>> 105: final Dimension size = getTextSize(font);
>>> 106: final BufferedImage image =
>>> 107: new BufferedImage(size.width, size.height, TYPE_3BYTE_BGR);
>>
>> t might not be related but we can test the image with alpha as well, in that case I think the aa might be always disabled and/or some other code path will be executed.
>>
>> The test for JDK-8204929 uses the "Locale.ENGLISH" and this one "Locale.US", does it matter which one is used to reproduce this bug?
>
>> It might not be related but we can test the image with alpha as well, in that case I think the aa might be always disabled and/or some other code path will be executed.
>
> This test covers the disabled AA and greyscale AA.
>
> Do you want to include both types of buffered images in the test? Do you want to switch to `TYPE_4BYTE_ABGR`? Do you want to use a translucent colour for rendering text?
>
>> The test for JDK-8204929 uses the "Locale.ENGLISH" and this one "Locale.US", does it matter which one is used to reproduce this bug?
>
> No, it doesn't matter here at all. Nothing would change if I didn't pass the locale, except that localised font names would be printed if a failure occurs.
>
> I'll update it to `Locale.ENGLISH` for consistency with `GlyphRotationTest.java` from JDK-8204929.
I think opaque+transparent images should be fine, do not think the actual format bgr/rgb/argb_pre/etc matters.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15335#discussion_r1297783638
More information about the client-libs-dev
mailing list