RFR: 8324243: Fix GCC 14 build

Phil Race prr at openjdk.org
Tue Jan 23 19:29:32 UTC 2024


On Tue, 23 Jan 2024 19:17:06 GMT, Sam James <duke at openjdk.org> wrote:

>> src/java.desktop/share/native/libharfbuzz/hb-object.hh line 328:
>> 
>>> 326:   if (unlikely (!user_data))
>>> 327:   {
>>> 328:     user_data = (hb_user_data_array_t *) hb_calloc (sizeof (hb_user_data_array_t), 1);
>> 
>> Note 
>> (1) We don't cherry pick unless it is critical.
>> (2) I doubt you have ownership of harfbuzz code such that you can contribute any part of it to JDK in accordance with the OCA you signed.
>> 
>> Accordingly, the right thing to do here is disable the warning for this library and tool chain.
>
> Sure, I wasn't exactly hiding that it was cherry-picked. I did not consider (2) to be an issue given OpenJDK wholesale imports releases, but w/e. (The fix is also trivial and essentially the only possible fix...)
> 
> Anyway, happy to do that. Can you suggest what the idiomatic way of doing that in the OpenJDK codebase is for imported libs?

add the warning here 
https://github.com/openjdk/jdk/blob/8b9bf758801400e4491326cd4c90fc117b9d97e1/make/modules/java.desktop/lib/Awt2dLibraries.gmk#L506

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17506#discussion_r1463888153


More information about the client-libs-dev mailing list