widcal: fix memory leak

Repeated redrawing would create multiple redraw-timeouts
espruino · Apr 18, 2022 · 4f17ed0 · 4f17ed0
1 parent dc273a7
commit 4f17ed0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/apps/widcal/ChangeLog b/apps/widcal/ChangeLog
@@ -1 +1,2 @@
-0.01: First version
+0.01: First version
+0.02: Fix memory leak
diff --git a/apps/widcal/metadata.json b/apps/widcal/metadata.json
@@ -1,7 +1,7 @@
 {
   "id": "widcal",
   "name": "Calendar Widget",
-  "version": "0.01",
+  "version": "0.02",
   "description": "Widget with the current date",
   "icon": "widget.png",
   "type": "widget",

diff --git a/apps/widcal/widget.js b/apps/widcal/widget.js
@@ -24,7 +24,8 @@
         ]);
       }
       // redraw when date changes
-      setTimeout(()=>WIDGETS["cal"].draw(), (86401 - Math.floor(date/1000) % 86400)*1000);
+      if (WIDGETS["cal"].to) clearTimeout(WIDGETS["cal"].to);
+      WIDGETS["cal"].to = setTimeout(()=>WIDGETS["cal"].draw(), (86401 - Math.floor(date/1000) % 86400)*1000);
     }
   };
 })();