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

Disable a LGTM warning, again (PR 13787 follow-up)

Browse Source 
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.
This commit is contained in:
Jonas Jenwald 2021-07-25 10:32:40 +02:00
parent aad0316137
commit 833f27c677
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/core/file_spec.js
View File

@ -66,8 +66,8 @@ class FileSpec {
get filename() { get filename() {
if (!this._filename && this.root) { if (!this._filename && this.root) {
const filename = pickPlatformItem(this.root) || "unnamed"; const filename = pickPlatformItem(this.root) || "unnamed";
this._filename = stringToPDFString(filename) this._filename = stringToPDFString(filename) // lgtm [js/double-escaping]
.replace(/\\\\/g, "\\") // lgtm [js/double-escaping] .replace(/\\\\/g, "\\")
.replace(/\\\//g, "/") .replace(/\\\//g, "/")
.replace(/\\/g, "/"); .replace(/\\/g, "/");
} }