RFR: 8320965: Scrolling on a touch enabled display fails on Wayland [v2]

Kevin Rushforth kcr at openjdk.org
Fri Dec 15 14:30:00 UTC 2023


On Thu, 14 Dec 2023 20:13:36 GMT, Jose Pereda <jpereda at openjdk.org> wrote:

>> modules/javafx.graphics/src/main/native-glass/gtk/glass_general.cpp line 599:
>> 
>>> 597:         return TRUE;
>>> 598:     }
>>> 599: #if GTK_CHECK_VERSION(3, 20, 0)
>> 
>> I wouldn't have expected any compile-time `#if` checks as part of this PR.
>
> Okay, that makes sense. 
> 
> I've removed the compile-time `#if-#else` from `wrapped.c`.
> 
> However, to do the same in `glass_general.cpp`, if the wrapped functions fail (when dlsym returns null), we still need to fall back to the old implementation, don't we? 
> 
> Therefore, I've changed the signature to return a `gboolean` (and removed the `GDK_GRAB_FAILED` enum value that was added in 3.16, as I just noticed).

This looks good. I'll review more closely and also test it.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1305#discussion_r1428032274


More information about the openjfx-dev mailing list