8067699: Better glyph storage
authorprr
Thu, 15 Jan 2015 09:47:06 -0800
changeset 29911 b2a9675e2e21
parent 29910 6691760b847e
child 29912 4478c08657e0
8067699: Better glyph storage Reviewed-by: srl, bae, mschoene
jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc.cpp
jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc2.cpp
--- a/jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc.cpp	Wed Jan 07 13:10:00 2015 -0800
+++ b/jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc.cpp	Thu Jan 15 09:47:06 2015 -0800
@@ -115,7 +115,7 @@
                   LE_DEBUG_BAD_FONT("off end of ligature substitution header");
                   return newState; // get out! bad font
               }
-              if(componentGlyph > glyphStorage.getGlyphCount()) {
+              if(componentGlyph >= glyphStorage.getGlyphCount()) {
                 LE_DEBUG_BAD_FONT("preposterous componentGlyph");
                 currGlyph++;
                 return newState; // get out! bad font
--- a/jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc2.cpp	Wed Jan 07 13:10:00 2015 -0800
+++ b/jdk/src/java.desktop/share/native/libfontmanager/layout/LigatureSubstProc2.cpp	Thu Jan 15 09:47:06 2015 -0800
@@ -119,7 +119,7 @@
 
             offset = action & lafComponentOffsetMask;
             if (offset != 0) {
-                if(componentGlyph > glyphStorage.getGlyphCount()) {
+                if(componentGlyph >= glyphStorage.getGlyphCount()) {
                   LE_DEBUG_BAD_FONT("preposterous componentGlyph");
                   currGlyph+= dir;
                   return nextStateIndex; // get out! bad font