RFR(XS): 8219675: Disable harfbuzz warnings with gcc 8

Mikael Vidstedt mikael.vidstedt at oracle.com
Tue Feb 26 00:09:55 UTC 2019

Erik/Phil, thanks for the reviews. Change pushed!

I also filed https://bugs.openjdk.java.net/browse/JDK-8219676 <https://bugs.openjdk.java.net/browse/JDK-8219676> to cover re-enabling the warnings (effectively reverting this change) once harfbuzz has been upgraded.


> On Feb 25, 2019, at 2:16 PM, Erik Joelsson <erik.joelsson at oracle.com> wrote:
> Looks good.
> /Erik
> On 2019-02-25 13:28, Mikael Vidstedt wrote:
>> Please review this change which (temporarily) disables two gcc warnings which are triggered by the harfbuzz code.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8219675 <https://bugs.openjdk.java.net/browse/JDK-8219675>
>> webrev: http://cr.openjdk.java.net/~mikael/webrevs/8219675/webrev.00/open/webrev/ <http://cr.openjdk.java.net/~mikael/webrevs/8219675/webrev.00/open/webrev/>
>> This change disables the two warnings in question (missing-attributes and class-memaccess). An alternative would be to make changes to the harfbuzz code itself instead.
>> Passes tier1.
>> Cheers,
>> Mikael

More information about the build-dev mailing list