From 833f27c677dbe4fd7ca72638e5cbe59362383aa6 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 25 Jul 2021 10:32:40 +0200 Subject: [PATCH] Disable a LGTM warning, again (PR 13787 follow-up) Apparently I didn't put one of the disable comments on the *correct* line, since I didn't read the instructions carefully enough, so let's try again. Note that, most unfortunately, disabling of warnings isn't applied until *after* a patch has been merged. --- src/core/file_spec.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/core/file_spec.js b/src/core/file_spec.js index cf06d66a3..7db874d5b 100644 --- a/src/core/file_spec.js +++ b/src/core/file_spec.js @@ -66,8 +66,8 @@ class FileSpec { get filename() { if (!this._filename && this.root) { const filename = pickPlatformItem(this.root) || "unnamed"; - this._filename = stringToPDFString(filename) - .replace(/\\\\/g, "\\") // lgtm [js/double-escaping] + this._filename = stringToPDFString(filename) // lgtm [js/double-escaping] + .replace(/\\\\/g, "\\") .replace(/\\\//g, "/") .replace(/\\/g, "/"); }