RFR: 8329471: Remove GTK2 [v4]
Alexander Zvegintsev
azvegint at openjdk.org
Wed Jul 31 19:17:05 UTC 2024
On Wed, 31 Jul 2024 18:29:57 GMT, Phil Race <prr at openjdk.org> wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> more cleanup
>
> src/java.desktop/share/classes/com/sun/java/swing/plaf/gtk/GTKPainter.java line 215:
>
>> 213: if (gtkState == SynthConstants.MOUSE_OVER) {
>> 214: if (GTKLookAndFeel.is3()) {
>> 215: paintComponentBackground(context, g, x, y, w, h);
>
> So do we really need the is3() check ?? Seems like not to me. Same for the case below at line 573.
It is not the only place, I doubted it, but now updated to remove all usages.
However, prefer to keep the `is3` method itself.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20378#discussion_r1698984338
More information about the build-dev
mailing list