RFR: 8249821: Separate libharfbuzz from libfontmanager

Erik Joelsson erik.joelsson at oracle.com
Tue Jul 21 22:03:28 UTC 2020


Hello Phil,

This looks pretty good, only some style nits.

Awt2dLibraries.gmk

428 and 466: These comments aren't relevant anymore now that harfbuzz is 
its own library.

434: Missed indent of 2

436: Too much indent (should be 2)

459-464: Too much indent

493-494: Too much indent

/Erik

On 2020-07-21 14:39, Philip Race wrote:
> Bug: https://bugs.openjdk.java.net/browse/JDK-8249821
> Webrev: http://cr.openjdk.java.net/~prr/8249821/
>
> This fix breaks out libharfbuzz from libfontmanager.
> As well as building I've done extensive testing on all platforms.
>
> I have tweaked the disabled warnings so we don't have un-needed 
> duplication
> but it is not a goal of this fix to resolve them.
> There's a bug (JDK-8074844) already filed to resolve them for 
> fontmanager and
> that should be easier to fix now.
>
> -phil.
>
>
>



More information about the build-dev mailing list