On Thu, 26 Jun 2025 14:18:53 GMT, Lukasz Kostyra <lkost...@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

Reply via email to