Fix a few *safe* ESLint no-var failures in src/core/evaluator.js (13371 follow-up)

As can be seen in PR 13371, some of the `no-var` changes in the `PartialEvaluator.{getOperatorList, getTextContent}` methods caused errors in `gulp server`-mode.
However, there's a handful of instances of `var` in other methods which should be completely *safe* to convert since there's no strange scope-issues present in that code.
This commit is contained in:
Jonas Jenwald 2021-05-16 15:15:12 +02:00
parent a5c74f53c1
commit 718f7bf7e1

View File

@ -394,7 +394,8 @@ class PartialEvaluator {
} else {
bbox = null;
}
let optionalContent = null;
let optionalContent = null,
groupOptions;
if (dict.has("OC")) {
optionalContent = await this.parseMarkedContentProps(
dict.get("OC"),
@ -404,7 +405,7 @@ class PartialEvaluator {
}
const group = dict.get("Group");
if (group) {
var groupOptions = {
groupOptions = {
matrix,
bbox,
smask,
@ -3766,7 +3767,7 @@ class PartialEvaluator {
throw new FormatError("invalid font name");
}
let fontFile;
let fontFile, subtype, length1, length2, length3;
try {
fontFile = descriptor.get("FontFile", "FontFile2", "FontFile3");
} catch (ex) {
@ -3778,13 +3779,13 @@ class PartialEvaluator {
}
if (fontFile) {
if (fontFile.dict) {
var subtype = fontFile.dict.get("Subtype");
if (subtype) {
subtype = subtype.name;
const subtypeEntry = fontFile.dict.get("Subtype");
if (subtypeEntry instanceof Name) {
subtype = subtypeEntry.name;
}
var length1 = fontFile.dict.get("Length1");
var length2 = fontFile.dict.get("Length2");
var length3 = fontFile.dict.get("Length3");
length1 = fontFile.dict.get("Length1");
length2 = fontFile.dict.get("Length2");
length3 = fontFile.dict.get("Length3");
}
}