RFR: JDK-8297230: Update Marlin2D to 0.9.4.6 [v4]

Laurent Bourgès lbourges at openjdk.org
Wed Nov 23 13:30:34 UTC 2022


On Wed, 23 Nov 2022 01:56:30 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Laurent Bourgès has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   fixed class header comment
>
> test/jdk/sun/java2d/marlin/HugePolygonClipTest.java line 68:
> 
>> 66:         System.out.println("HugePolygonClipTest: size = " + SCENE_WIDTH + " x " + SCENE_HEIGHT);
>> 67: 
>> 68:         final BufferedImage image = new BufferedImage(SCENE_WIDTH, SCENE_HEIGHT, BufferedImage.TYPE_INT_ARGB);
> 
> It looks fine. One additional question did we have any tests to validate how the "custom images" will work? For example the image similar to this [one](https://github.com/openjdk/jdk/blob/master/src/java.desktop/share/classes/java/awt/image/BufferedImage.java#L383) but using a different order of offsets, like: int[] bOffs = {1, 0, 2, 3}; For some other bug I have found that some our code does not expect "non-default" layouts.

Thanks.
In this particular test I selected ARGB as a well-known format so it does not work with non-standard ARGB channels. I could have used getRed(), getGreen() values to use standard API to get each components.

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

PR: https://git.openjdk.org/jdk/pull/11225



More information about the client-libs-dev mailing list