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

The first glyph in CFF CIDFonts must be named 0 instead of ".notdef"

Browse Source 
Fixes #11718 in which the `ff` ligature glyph is at index zero in a CFF font. Beacuse this is a CIDFont, glyph names are CIDs, which are integers. Thus the string `".notdef"` is not correct. The rest of the charset data is already parsed correctly as integers when the boolean argument `cid` is true.
This commit is contained in:
Jani Pehkonen 2020-03-24 15:33:43 +02:00
parent 85838fc505
commit a22c0eab48
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

@ -268,7 +268,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() {
@ -286,7 +286,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() {