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

Simplify JpxImage.setOptions a little bit

Browse Source 
After PR 19392 we're only invoking this method *once* per document, hence the early-return branch shouldn't be necessary any more.
This commit is contained in:
Jonas Jenwald 2025-02-22 13:38:10 +01:00
parent 641e2f506e
commit 637e95985a
2 changed files with 1 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/core/jpx.js
View File

@ -38,9 +38,6 @@ class JpxImage {
static #wasmUrl = null; static #wasmUrl = null;
static setOptions({ handler, useWasm, useWorkerFetch, wasmUrl }) { static setOptions({ handler, useWasm, useWorkerFetch, wasmUrl }) {
if (this.#buffer || this.#modulePromise) {
return;
}
this.#useWasm = useWasm; this.#useWasm = useWasm;
this.#useWorkerFetch = useWorkerFetch; this.#useWorkerFetch = useWorkerFetch;
this.#wasmUrl = wasmUrl; this.#wasmUrl = wasmUrl;

2
src/core/pdf_manager.js
View File

@ -70,7 +70,7 @@ class BasePdfManager {
FeatureTest.isImageDecoderSupported; FeatureTest.isImageDecoderSupported;
this.evaluatorOptions = Object.freeze(evaluatorOptions); this.evaluatorOptions = Object.freeze(evaluatorOptions);
// Initially image-options once per document. // Initialize image-options once per document.
ImageResizer.setOptions(evaluatorOptions); ImageResizer.setOptions(evaluatorOptions);
JpegStream.setOptions(evaluatorOptions); JpegStream.setOptions(evaluatorOptions);
JpxImage.setOptions({ ...evaluatorOptions, handler }); JpxImage.setOptions({ ...evaluatorOptions, handler });