From 70d362f22ca8096b7303dfa09a9a294321804f31 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald <jonas.jenwald@gmail.com> Date: Thu, 17 Nov 2022 15:19:40 +0100 Subject: [PATCH] Remove an unnecessary variable in `getPdfManager`, in the `src/core/worker.js` file Another tiny piece of clean-up, since adding a `catch`-handler to a Promise shouldn't require an intermediate variable. --- src/core/worker.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/core/worker.js b/src/core/worker.js index 3f8c73379..1e9a807ec 100644 --- a/src/core/worker.js +++ b/src/core/worker.js @@ -303,7 +303,7 @@ class WorkerMessageHandler { } cachedChunks = []; }; - const readPromise = new Promise(function (resolve, reject) { + new Promise(function (resolve, reject) { const readChunk = function ({ value, done }) { try { ensureNotTerminated(); @@ -335,8 +335,7 @@ class WorkerMessageHandler { } }; fullRequest.read().then(readChunk, reject); - }); - readPromise.catch(function (e) { + }).catch(function (e) { pdfManagerCapability.reject(e); cancelXHRs = null; });