On Mon, 3 Oct 2022 22:43:40 GMT, Harshitha Onkar <[email protected]> wrote:
>> src/java.desktop/share/classes/javax/swing/plaf/metal/MetalBorders.java line
>> 329:
>>
>>> 327:
>>> 328: // Draw the bulk of the border
>>> 329: for (int i = 1; i <= loopCount; i++) {
>>
>> Should we redo the drawing of this border to use Graphics2D fillRect instead?
>
> That was my initial thought too. But, as the loopCount won't be really large
> (Eg. it is 15 for 300% scaling) and it wouldn't affect the performance
> drastically, I have retained the original approach to draw the bulk of the
> border.
Let us leave it as is at this time. There are enough changes.
At the same time, it's a good suggestion for optimization. We should consider
it in a separate CR.
-------------
PR: https://git.openjdk.org/jdk/pull/10274
