fix: scoping issue on phase variable (#1557)

I also have an open PR to fontkit. foliojs/fontkit#267
diegomura · Nov 7, 2021 · b536a2e · b536a2e
1 parent 24ca9d4
commit b536a2e
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/fontkit/src/glyph/CFFGlyph.js b/packages/fontkit/src/glyph/CFFGlyph.js
@@ -320,8 +320,8 @@ export default class CFFGlyph extends Glyph {
               break;
 
             case 30: // vhcurveto
-            case 31: // hvcurveto
-              phase = op === 31;
+            case 31: { // hvcurveto
+              let phase = op === 31;
               while (stack.length >= 4) {
                 if (phase) {
                   c1x = x + stack.shift();
@@ -343,6 +343,7 @@ export default class CFFGlyph extends Glyph {
                 phase = !phase;
               }
               break;
+            }
 
             case 12:
               op = stream.readUInt8();