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 <[email protected]
<mailto:[email protected]>> 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.
