Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-08-06 22:37:23 +02:00
Code Issues Packages Projects Releases Wiki Activity

Cache the entire scand regular expression, in src/scripting_api/util.js

Browse Source 
Rather than only caching a string and then re-creating the regular expression on every `Util.prototype._scand` invocation, the entire regular expression can be cached directly instead.
This commit is contained in:
Jonas Jenwald 2026-08-04 11:46:47 +02:00
parent f1f1782f38
commit 59cd4568b2
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/scripting_api/util.js
View File

@ -602,7 +602,7 @@ class Util extends PDFObject {
: pattern; : pattern;
}); });
return [re, actions]; return [new RegExp(`^${re}$`, "g"), actions];
} }
_scand(cFormat, cDate, strict = false) { _scand(cFormat, cDate, strict = false) {
@ -622,12 +622,10 @@ class Util extends PDFObject {
return this.scand("m/d/yy h:MM:ss tt", cDate); return this.scand("m/d/yy h:MM:ss tt", cDate);
} }
const [re, actions] = (this.#scandCache ??= new Map()).getOrInsertComputed( const [regex, actions] = (this.#scandCache ??=
cFormat, new Map()).getOrInsertComputed(cFormat, this.#createScandDataBound);
this.#createScandDataBound
);
const matches = new RegExp(`^${re}$`, "g").exec(cDate); const matches = regex.exec(cDate);
if (!matches || matches.length !== actions.length + 1) { if (!matches || matches.length !== actions.length + 1) {
return strict ? null : this.#tryToGuessDate(cFormat, cDate); return strict ? null : this.#tryToGuessDate(cFormat, cDate);
} }