Merge pull request #8243 from brendandahl/no-skip-glyph0

Don’t skip glyph 0 in cmap.
8 years ago · a662d26814
3 changed files with 6 additions and 3 deletions
--- a/src/core/fonts.js
+++ b/src/core/fonts.js
@ -1474,9 +1474,6 @@ var Font = (function FontClosure() {
				@@ -1474,9 +1474,6 @@ var Font = (function FontClosure() {
              glyphId = (offsetIndex < 0 ?
                         j : offsets[offsetIndex + j - start]);
              glyphId = (glyphId + delta) & 0xFFFF;
-              if (glyphId === 0) {
-                continue;
-              }
              mappings.push({
                charCode: j,
                glyphId: glyphId
--- a/test/pdfs/issue8234.pdf
+++ b/test/pdfs/issue8234.pdf
--- a/test/test_manifest.json
+++ b/test/test_manifest.json
@ -1243,6 +1243,12 @@
				@@ -1243,6 +1243,12 @@
      "rounds": 1,
      "type": "eq"
    },
+    {  "id": "issue8234",
+      "file": "pdfs/issue8234.pdf",
+      "md5": "32650fc60c51a9813b98bc9876dc15af",
+      "rounds": 1,
+      "type": "eq"
+    },
    {  "id": "type4psfunc",
      "file": "pdfs/type4psfunc.pdf",
      "md5": "7e6027a02ff78577f74dccdf84e37189",