Looks fine to me so long as the build is happy.

-phil.

On 2/25/19 1:28 PM, 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
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


Reply via email to