On Mon, 12 Dec 2022 19:09:02 GMT, Harshitha Onkar <hon...@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

Reply via email to