Re: Review Request Re: [PATCH] Fix bug to handle minus value glyph_id on Windows

2018-07-24 Thread Kevin Rushforth
As an FYI, this is filed in JBS as: https://bugs.openjdk.java.net/browse/JDK-8207839 and is now being reviewed. -- Kevin On 7/5/2018 8:10 PM, Nakajima Akira wrote: Hi All. I created github account today  and re-posted this patch to github. Please review the following fix: patch: https://git

Review Request Re: [PATCH] Fix bug to handle minus value glyph_id on Windows

2018-07-05 Thread Nakajima Akira
Hi All. I created github account today and re-posted this patch to github. Please review the following fix: patch: https://github.com/javafxports/openjdk-jfx/pull/125 Regards, Akira Nakajima -- Company: NTT Comware Corporation Name: Akira Nakajima E-Mail: n

Re: [PATCH] Fix bug to handle minus value glyph_id on Windows

2018-06-26 Thread Nakajima Akira
This bug is happened on Windows. Sorry, I should use SHORTMASK instead of intMask. I re-send patch. Difference is name only. intMask -> SHORTMASK diff --git a/modules/javafx.graphics/src/main/java/com/sun/javafx/font/directwrite/DWGlyph.java b/modules/javafx.graphics/src/main/java/com/sun/

[PATCH] Fix bug to handle minus value glyph_id on Windows

2018-06-26 Thread Nakajima Akira
# This patch is separated from http://mail.openjdk.java.net/pipermail/openjfx-dev/2018-June/022005.html Java issue error of ArrayIndexOutOfBoundsException by run following Test program on Windows. This bug is that Java handles big glyph_id as minus value. For Example, Inside JavaFX, glyph_id