Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #11744 from janpe2/cff-glyph-zero

Browse Source 
The first glyph in CFF CIDFonts must be named 0 instead of ".notdef"
This commit is contained in:
Tim van der Meij 2020-03-24 23:52:21 +01:00 committed by GitHub
commit 475fa1f97f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 10 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/core/cff_parser.js
View File

@ -803,7 +803,7 @@ var CFFParser = (function CFFParserClosure() {
var bytes = this.bytes;
var start = pos;
var format = bytes[pos++];
var charset = [".notdef"];
const charset = [cid ? 0 : ".notdef"];
var id, count, i;
// subtract 1 for the .notdef glyph

1
test/pdfs/.gitignore vendored
View File

@ -254,6 +254,7 @@
!bug903856.pdf
!bug850854.pdf
!bug866395.pdf
!issue11718_reduced.pdf
!bug1027533.pdf
!bug1028735.pdf
!bug1046314.pdf

BIN
test/pdfs/issue11718_reduced.pdf Normal file
View File

Binary file not shown.

6
test/test_manifest.json
View File

@ -3339,6 +3339,12 @@
"rounds": 1,
"type": "eq"
},
{ "id": "issue11718",
"file": "pdfs/issue11718_reduced.pdf",
"md5": "a0deea064b4171bb8ea9f6e8a523e594",
"rounds": 1,
"type": "eq"
},
{ "id": "issue3061",
"file": "pdfs/issue3061.pdf",
"md5": "696a7cb1b194d095ca3f7861779a606b",

4
test/unit/cff_parser_spec.js
View File

@ -276,7 +276,7 @@ describe("CFFParser", function() {
// CID font
charset = parser.parseCharsets(3, 2, new CFFStrings(), true);
expect(charset.charset).toEqual([".notdef", 8, 9]);
expect(charset.charset).toEqual([0, 8, 9]);
});
it("parses charset format 2", function() {
@ -294,7 +294,7 @@ describe("CFFParser", function() {
// CID font
charset = parser.parseCharsets(3, 2, new CFFStrings(), true);
expect(charset.charset).toEqual([".notdef", 8, 9]);
expect(charset.charset).toEqual([0, 8, 9]);
});
it("parses encoding format 0", function() {