RFR: JDK-8307604: gcc12 based Alpine build broken build after JDK-8307301

[Lutz Schmidt]

On Mon, 8 May 2023 10:29:56 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:

> After the harfbuzz 7.2 update we run into
> 
> /linuxmuslx86_64/jdk-dev/src/java.desktop/share/native/libharfbuzz/OT/glyf/Glyph.hh:281:8: error: offset '4' outside bounds of constant string [-Werror=array-bounds]
>   281 | bool get_points (hb_font_t *font, const accelerator_t &glyf_accelerator,
>       | ^~~~~~~~~~
> /linuxmuslx86_64/jdk-dev/src/java.desktop/share/native/libharfbuzz/hb-static.cc:45:16: note: '_hb_NullPool' declared here
>    45 | uint64_t const _hb_NullPool[(HB_NULL_POOL_SIZE + sizeof (uint64_t) - 1) / sizeof (uint64_t)] = {};
>       | ^~~~~~~~~~~~
> /linuxmuslx86_64/jdk-dev/src/java.desktop/share/native/libharfbuzz/OT/glyf/Glyph.hh:281:8: error: offset '4' outside bounds of constant string [-Werror=array-bounds]
>   281 | bool get_points (hb_font_t *font, const accelerator_t &glyf_accelerator,
>       | ^~~~~~~~~~
> /linuxmuslx86_64/jdk-dev/src/java.desktop/share/native/libharfbuzz/hb-static.cc:45:16: note: '_hb_NullPool' declared here
>    45 | uint64_t const _hb_NullPool[(HB_NULL_POOL_SIZE + sizeof (uint64_t) - 1) / sizeof (uint64_t)] = {};
> 
> We use gcc12 on Alpine.
> Switching off this  'warning as error' fixes the issue.

LGTM.
Disabling the warning is OK for now. Issue should be fixed by HarfBuzz.

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

Marked as reviewed by lucy (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13859#pullrequestreview-1416631184