Re: [OpenJDK 2D-Dev] RFR: 8183978 : Remove ICU layout code from OpenJDK

2017-10-12 Thread Sergey Bylokhov
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,

Re: [OpenJDK 2D-Dev] RFR: 8183978 : Remove ICU layout code from OpenJDK

2017-10-12 Thread Steven R. Loomis
approve On Wed, Oct 11, 2017 at 9:59 AM, Phil Race wrote: > Bug: https://bugs.openjdk.java.net/browse/JDK-8183978 > Webrev: 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