RFR: 8339364: AIX build fails: various unused variable and function warnings [v2]

[Matthias Baesken]

On Mon, 2 Sep 2024 13:25:51 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

>> We get a couple of warnings as errors on AIX because of unused variables or functions , for example :
>> /priv/jenkins/client-home/workspace/openjdk-jdk-dev-aix_ppc64-opt/jdk/src/java.base/unix/native/libjava/ProcessHandleImpl_unix.c:665:10: error: unused variable 'exePath' [-Werror,-Wunused-variable]
>>     char exePath[PATH_MAX];
>>          ^
>> /priv/jenkins/client-home/workspace/openjdk-jdk-dev-aix_ppc64-opt/jdk/src/java.base/unix/native/libjava/ProcessHandleImpl_unix.c:668:9: error: unused variable 'ret' [-Werror,-Wunused-variable]
>>     int ret;
>>         ^
>> /priv/jenkins/client-home/workspace/openjdk-jdk-dev-aix_ppc64-opt/jdk/src/java.base/unix/native/libjava/ProcessHandleImpl_unix.c:664:10: error: unused variable 'fn' [-Werror,-Wunused-variable]
>>     char fn[32];
>>          ^
>> 
>> This seems to be related to the recent make changes
>> 8339156: Use more fine-granular clang unused warnings
>> https://bugs.openjdk.org/browse/JDK-8339156
>> (we use clang too on AIX)
>
> Matthias Baesken has updated the pull request incrementally with one additional commit since the last revision:
> 
>   adjust indentation in X11Color.c

I would suggest creating a JBS issue and later PR for the pthread_create return code checking (both unix and macos).

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

PR Comment: https://git.openjdk.org/jdk/pull/20812#issuecomment-2325785214