From efe3cb2982855116952628758a6424328c2dd826 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 13 May 2022 17:12:03 +0200 Subject: [PATCH] Remove unnecessary clean-up in `Toolbar.#adjustScaleWidth` While zeroing the temporary `canvas` makes sense, manually clearing the canvas and its context doesn't really accomplish anything since those are tied to the scope of the method. --- web/toolbar.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/web/toolbar.js b/web/toolbar.js index 93a39a56d..c0916fc76 100644 --- a/web/toolbar.js +++ b/web/toolbar.js @@ -260,8 +260,8 @@ class Toolbar { ); // The temporary canvas is used to measure text length in the DOM. - let canvas = document.createElement("canvas"); - let ctx = canvas.getContext("2d", { alpha: false }); + const canvas = document.createElement("canvas"); + const ctx = canvas.getContext("2d", { alpha: false }); ctx.font = `${style.fontSize} ${style.fontFamily}`; let maxWidth = 0; @@ -281,7 +281,6 @@ class Toolbar { // immediately, which can greatly reduce memory consumption. canvas.width = 0; canvas.height = 0; - canvas = ctx = null; } }