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