I believe
668 /* See the comments in getGlyphMetricsNative. They apply here too.
*/ should be
getGlyphAdvanceNative
Other than that, looks good to me.
Regards
Prasanta
On 03-Nov-19 7:31 PM, Jayathirth Rao wrote:
Changes are fine.
Thanks,
Jay
On 01-Nov-2019, at 1:29 AM, Sergey Bylokhov
Changes are fine.
Thanks,
Jay
> On 01-Nov-2019, at 1:29 AM, Sergey Bylokhov
> wrote:
>
> Looks fine.
>
> On 10/30/19 11:37 am, Phil Race wrote:
>> PS .. a positive from this is that with this fix the regression test
>> runs about 10X faster than it did with say JDK 13 GA .. due to
>>
Looks fine.
On 10/30/19 11:37 am, Phil Race wrote:
PS .. a positive from this is that with this fix the regression test
runs about 10X faster than it did with say JDK 13 GA .. due to
skipping the redundant images
-phil.
On 10/30/19 11:05 AM, Phil Race wrote:
Bug:
PS .. a positive from this is that with this fix the regression test
runs about 10X faster than it did with say JDK 13 GA .. due to
skipping the redundant images
-phil.
On 10/30/19 11:05 AM, Phil Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8233097
Webrev:
Bug: https://bugs.openjdk.java.net/browse/JDK-8233097
Webrev: http://cr.openjdk.java.net/~prr/8233097/
This bug is a regression from a fix in JDK 13.0.1 / 11.0.5
To work around a bug in old versions of freetype we capped the size
of the glyph image but overlooked that we may use that code to
