* implement IHash for js Symbol

* add test cases
clojure · Feb 21, 2024 · 4defb9e · 4defb9e
1 parent 4f400cb
commit 4defb9e
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/main/cljs/cljs/core.cljs b/src/main/cljs/cljs/core.cljs
@@ -1458,6 +1458,11 @@
   (-hash [o]
     (goog/getUid o)))
 
+(extend-type symbol
+  IHash
+  (-hash [o]
+    (hash (.toString o))))
+
 ;;this is primitive because & emits call to array-seq
 (defn inc
   "Returns a number one greater than num."

diff --git a/src/test/cljs/cljs/hashing_test.cljs b/src/test/cljs/cljs/hashing_test.cljs
@@ -102,3 +102,11 @@
     (is (= (hash 0.5) (hash 0.5)))
     (is (= (hash -0.32553251) (hash -0.32553251)))
     (is (= (hash -0.0000032553251) (hash -0.0000032553251)))))
+
+(deftest test-cljs-3290
+  (testing "JS Symbol hash"
+    (let [s (.for js/Symbol "foo")]
+      (is (number? (hash s)))
+      (is (== (hash s) (hash s)))
+      (let [m {s 2}]
+        (is (== 2 (get m s)))))))