From 99b6d3ea35b12ba700a0a9f4f154bfe60b7ec0ea Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 16 May 2021 15:15:12 +0200 Subject: [PATCH] Fix a few *safe* ESLint `no-var` failures (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. --- src/core/evaluator.js | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/core/evaluator.js b/src/core/evaluator.js index b708825d4b60d0..59791cb9050827 100644 --- a/src/core/evaluator.js +++ b/src/core/evaluator.js @@ -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"); } }