On Tue, 11 Feb 2025 08:33:09 GMT, Julian Waters <jwat...@openjdk.org> wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one >> additional commit since the last revision: >> >> move fix to spa/utils/endian.h > > src/java.desktop/unix/native/libpipewire/include/spa/param/audio/raw.h line > 16: > >> 14: #if !defined(AIX) >> 15: #include <spa/utils/endian.h> >> 16: #endif > > The ifdef conditional blocks could be merged if I am not mistaken? Yes, at first I kept it as it was in the original fix. But after revisiting it, I think that it is belongs in the new `spa/utils/endian.h` file. @MBaesken could you please recheck if it still works for you? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23543#discussion_r1950694146
