On Mon, 2 Sep 2024 20:06:06 GMT, Martin Doerr <mdo...@openjdk.org> wrote:
>> Matthias Baesken has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> adjust indentation in X11Color.c
>
> src/java.desktop/unix/native/common/awt/X11Color.c line 1234:
>
>> 1232: awt_allocate_systemrgbcolors (jint *rgbColors, int num_colors,
>> 1233: AwtGraphicsConfigDataPtr awtData) {
>> 1234: for (int i = 0; i < num_colors; i++)
>
> This requires C99. Is this ok for all platforms with all supported compilers?
Yes, we have C11 as minimum for the JDK.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20812#discussion_r1741270798
