On Mon, 4 May 2026 17:03:47 GMT, Alexander Zvegintsev <[email protected]>
wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8383605
>
> test/jdk/java/awt/image/Raster/InterleavedRasterTest.java line 39:
>
>> 37: int h = 1;
>> 38: int b = -1;
>> 39: try {
>
> The indentation is inconsistent.
fixed
> test/jdk/java/awt/image/Raster/InterleavedRasterTest.java line 41:
>
>> 39: try {
>> 40: Raster.createInterleavedRaster(DataBuffer.TYPE_BYTE, w, h, b,
>> null);
>> 41: } catch (IllegalArgumentException e) {
>
> This test does not verify whether an `IllegalArgumentException` is thrown.
>
> I guess it would be better to extract such check into a method that only
> returns if IAE is thrown and throws a runtime exception in other cases.
ok
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/31014#discussion_r3184561970
PR Review Comment: https://git.openjdk.org/jdk/pull/31014#discussion_r3184561574
