RFR: JDK-8294680: Refactor scaled border rendering

Rajat Mahajan duke at openjdk.org
Mon Dec 12 20:45:57 UTC 2022


On Mon, 12 Dec 2022 19:09:02 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>> src/java.desktop/share/classes/com/sun/java/swing/SwingUtilities3.java line 190:
>> 
>>> 188: 
>>> 189:         if (resetTransform) {
>>> 190:             double xx = at.getScaleX() * x + at.getTranslateX();
>> 
>> Can we combine this code block with the above if (resetTransform) at line 173 ? or did you keep it separate for a reason?
>
> @rajamah In-order to paint the border even when g is not a Graphics2D object I have added the `resetTransform` check here again and additionally to keep the if & else block close-by, which is easier to understand rather than having the blocks separated.

sure, makes sense. Thanks

-------------

PR: https://git.openjdk.org/jdk/pull/11571



More information about the client-libs-dev mailing list