RFR: 8279614: The left line of the TitledBorder is not painted on 150 scale factor [v3]

[Phil Race]

On Mon, 14 Mar 2022 20:57:26 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Alisen Chung has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   added functions for drawing border, fixed translate
>
> src/java.desktop/share/classes/javax/swing/border/EtchedBorder.java line 154:
> 
>> 152: 
>> 153:         // 8279614: The dark line was being overdrawn by the light line. The fix was to
>> 154:         // make sure that the dark line is always drawn second.
> 
> Suggestion:
> 
>         // 8279614: The dark line was overdrawn by the light line.
>         // The fix makes sure that the dark line is always drawn second.
> 
> 
> Does the opposite never happen? That the shadow overdraws the highlight.

This is a comment on the actual comment rather than whether the intent is correct.
I'm not in favour of putting bug ids into the comments. Our code will end up utterly cluttered with them
Also it is phrased to describe a fix.
So IF it were to stand as is, I would have phrased it as "what we are doing now and why", ie
// Make sure the shadow line always is drawn second because it needs to over draw the border.

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

PR: https://git.openjdk.java.net/jdk/pull/7449