RFR: JDK-8015739: Background of JInternalFrame is located out of JInternalFrame [v4]

Alexey Ivanov aivanov at openjdk.org
Tue Oct 4 18:40:24 UTC 2022


On Mon, 3 Oct 2022 22:43:40 GMT, Harshitha Onkar <honkar at 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



More information about the client-libs-dev mailing list