RFR: 8317696: Fix compilation with clang-16 [v3]
Jan Kratochvil
jkratochvil at openjdk.org
Mon Oct 9 15:01:45 UTC 2023
On Mon, 9 Oct 2023 14:38:27 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
>> Jan Kratochvil has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove declaration changes which are not required
>> - suggested by djelinski
>
> src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.c line 218:
>
>> 216: static GtkWidget* (*fp_gtk_menu_bar_new)();
>> 217: static GtkWidget* (*fp_gtk_menu_item_new)();
>> 218: static GtkWidget* (*fp_gtk_notebook_new)(void);
>
> This change is inconsistent with the surrounding style; I'd keep the original declaration, and only remove the NULL in function calls.
I did not notice it is not required to change the prototype. OK, it works without this change.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16092#discussion_r1350418413
More information about the net-dev
mailing list