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.

Reply via email to