On Mon, 3 Oct 2022 22:43:40 GMT, Harshitha Onkar <hon...@openjdk.org> 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

Reply via email to