Looks fine.
I guess we should update TestLayoutVsICU.java test which was added as
part of harfbuzz integration. I assumed that it should compares behavior
against icu, but it seems it is never executed.
On 12/10/2017 10:45, Steven R. Loomis wrote:
approve
On Wed, Oct 11, 2017 at 9:59 AM, Phil Race <philip.r...@oracle.com
<mailto:philip.r...@oracle.com>> wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8183978
<https://bugs.openjdk.java.net/browse/JDK-8183978>
Webrev: http://cr.openjdk.java.net/~prr/8183978/
<http://cr.openjdk.java.net/~prr/8183978/>
This fix removes the obsolete ICU opentype layout code from JDK 10.
I've built this on all platforms (including open+closed builds).
-phil.
--
Best regards, Sergey.