fix(lru): use map.delete() directly (#713)

The default implementation already returns boolean if the value has been deleted. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/delete#return_value It's also faster since we don't double check the hashmap for a value. --------- Co-authored-by: Luke Karrys <[email protected]>
npm · May 9, 2024 · 6466ba9 · 6466ba9
1 parent d777418
commit 6466ba9
Showing 1 changed file with 1 addition and 6 deletions.
diff --git a/internal/lrucache.js b/internal/lrucache.js
@@ -17,12 +17,7 @@ class LRUCache {
   }
 
   delete (key) {
-    if (this.map.has(key)) {
-      this.map.delete(key)
-      return true
-    } else {
-      return false
-    }
+    return this.map.delete(key)
   }
 
   set (key, value) {