On Fri, 14 Mar 2025 02:02:58 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> make/modules/java.desktop/lib/ClientLibraries.gmk line 249:
>> 
>>> 247:       DISABLED_WARNINGS_clang_dgif_lib.c := sign-compare, \
>>> 248:       DISABLED_WARNINGS_clang_gzwrite.c := format-nonliteral, \
>>> 249:       DISABLED_WARNINGS_clang_png.c := unused-function, \
>> 
>> This warning was disabled to fix the following build issue on linux and 
>> macOS.
>> 
>> 
>> open/src/java.desktop/share/native/libsplashscreen/libpng/png.c:1558:1: 
>> error: 'png_icc_profile_error' defined but not used [-Werror=unused-function]
>> [2025-03-12T23:57:44,575Z]  1558 | png_icc_profile_error(png_const_structrp 
>> png_ptr, png_const_charp name,
>> [2025-03-12T23:57:44,575Z]       | ^~~~~~~~~~~~~~~~~~~~~
>
> Might this be a bug in PNG? All usages of this function are now guarded by 
> "PNG_READ_iCCP_SUPPORTED"? It would be useful to report this upstream.

@mrserb I see, what you meant is that the function `png_icc_profile_error` 
should be guarded with PNG_READ_iCCP_SUPPORTED instead of PNG_iCCP_SUPPORTED. 
Reported it to upstream - https://github.com/pnggroup/libpng/issues/668

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24021#discussion_r1996304716

Reply via email to