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

Optimize convertOpList in svg.js by computing the operator ID mapping only once

Browse Source 
There is no need to recompute this for every operator list we encounter.
This commit is contained in:
Tim van der Meij 2019-04-06 16:52:29 +02:00
parent 2b18e5a355
commit 5a03b1c0d7
No known key found for this signature in database
GPG Key ID: 8C3FD2925A5F2762
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/display/svg.js
View File

@ -432,6 +432,14 @@ SVGGraphics = class SVGGraphics {
this.embeddedFonts = Object.create(null);
this.cssStyle = null;
this.forceDataSchema = !!forceDataSchema;
// In `src/shared/util.js` the operator names are mapped to IDs.
// The list below represents the reverse of that, i.e., it maps IDs
// to operator names.
this._operatorIdMapping = [];
for (const op in OPS) {
this._operatorIdMapping[OPS[op]] = op;
}
}
save() {
@ -493,11 +501,7 @@ SVGGraphics = class SVGGraphics {
}
convertOpList(operatorList) {
const REVOPS = [];
for (const op in OPS) {
REVOPS[OPS[op]] = op;
}
const operatorIdMapping = this._operatorIdMapping;
const argsArray = operatorList.argsArray;
const fnArray = operatorList.fnArray;
const opList = [];
@ -505,7 +509,7 @@ SVGGraphics = class SVGGraphics {
const fnId = fnArray[i];
opList.push({
'fnId': fnId,
'fn': REVOPS[fnId],
'fn': operatorIdMapping[fnId],
'args': argsArray[i],
});
}