RFR: 8357584: [XWayland] [OL10] Robot.mousePress() is delivered to wrong place

Thu Jun 26 17:05:35 UTC 2025

On Thu, 26 Jun 2025 14:18:53 GMT, Lukasz Kostyra <lkostyra at openjdk.org> wrote:

> These PipeWire declarations have been just moved here from the original header. Is this move necessary?

It is about clang 17 warning, it was [fixed](https://github.com/openjdk/jdk/pull/23236) on the jdk side.
I am trying to keep the code similar between OpenJDK and OpenJFX, so I brought this fix here to make maintenance easier.

> Also (a general thought towards past implementation, not just to your change) I think we should not define those manually at all. There are PipeWire headers defining those that we should include (and I believe we do already include some in screencast_pipewire.h

The point is basically the same: ease of maintenance between OpenJDK and OpenJFX.
OpenJDK can also be built on very old Linux distributions that lack the necessary headers.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1834#discussion_r2169505361