<div dir="ltr">no you have to revert the commit<br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 17 janv. 2023 à 17:55, Alan Bateman <<a href="mailto:alanb@openjdk.org">alanb@openjdk.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, 17 Jan 2023 13:33:06 GMT, Kevin Rushforth <<a href="mailto:kcr@openjdk.org" target="_blank">kcr@openjdk.org</a>> wrote:<br>
<br>
>> Hi all,<br>
>> <br>
>> Please review the fix for the build failure with clang-15.<br>
>> <br>
>> 1. -Wbitwise-instead-of-logical<br>
>> <br>
>>    1) src/hotspot/share/oops/generateOopMap.cpp          <--- fixed the warning<br>
>>    2) src/hotspot/share/runtime/notificationThread.cpp   <--- keep the code and disable warnings<br>
>>    3) src/hotspot/share/runtime/serviceThread.cpp        <--- keep the code and disable warnings<br>
>> <br>
>> <br>
>> 2. -Wdeprecated-non-prototype (all the warnings are disabled)<br>
>> <br>
>>    1) Mainly caused by files under `src/java.base/share/native/libzip/zlib/`  <--- keep the code and disable warnings<br>
>>       It occurred while building LIBJLI, LIBZIP and LIBSPLASHSCREEN.<br>
>> <br>
>>    2) While compiling src/java.desktop/share/native/libawt/awt/medialib/awt_ImagingLib.c  <--- keep the code and disable warnings<br>
>> <br>
>>    3) Caused by src/java.desktop/macosx/native/libawt_lwawt/awt/CPrinterJob.m  <--- fixed the warnings<br>
>> <br>
>> <br>
>> 3. -Wdeprecated-builtins<br>
>> <br>
>>    Caused by files under src/java.desktop/share/native/libharfbuzz/  <--- fixed the warnings<br>
>> <br>
>>    Ref: <a href="https://github.com/harfbuzz/harfbuzz/blob/main/src/hb-meta.hh#L202" rel="noreferrer" target="_blank">https://github.com/harfbuzz/harfbuzz/blob/main/src/hb-meta.hh#L202</a><br>
>> <br>
>> 4. -Wgnu-folding-constant<br>
>> <br>
>>    Caused by src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLRenderer.m  <--- keep the code and disable warnings<br>
>> <br>
>> <br>
>> Thanks.<br>
>> Best regards,<br>
>> Jie<br>
><br>
> Yes, but I would wait for @prrace to review the Harfbuzz change.<br>
<br>
> Hi @kevinrushforth and @AlanBateman , are you fine with this change?<br>
<br>
No objection from me but I agree with Kevin to wait to see what Phil says about Harfbuzz.<br>
<br>
-------------<br>
<br>
PR: <a href="https://git.openjdk.org/jdk/pull/12005" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/12005</a><br>
</blockquote></div>