On Mon, 18 Mar 2024 08:04:36 GMT, Abhishek Kumar <abhis...@openjdk.org> wrote:
>> Tejesh R has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Review updates
>
> test/jdk/javax/swing/border/Test4129681.java line 62:
>
>> 60: label.setSize(SIZE);
>> 61: label.setEnabled(false);
>> 62: BufferedImage image = new BufferedImage((int)
>> Math.ceil(label.getWidth()),
>
> It can be -
> BufferedImage(label.getWidth(), label.getHeight(),
> TYPE_INT_RGB);
>
> Why Math.ceil is required ?
Removed.
> test/jdk/javax/swing/border/Test4129681.java line 79:
>
>> 77: }
>> 78:
>> 79: if (((double)correctColoredPixels/totalPixels*100) <= tolerance)
>> {
>
> fix spacing.
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18336#discussion_r1528218032
PR Review Comment: https://git.openjdk.org/jdk/pull/18336#discussion_r1528217624
