java.util.Map for defrecord, #313

Signed-off-by: Stuart Halloway <[email protected]>

Author: stuarthalloway

src/clj/clojure/core_deftype.clj

@@ -196,8 +196,23 @@
                    `(without [~'this ~'k] (if (contains? #{~@(map keyword base-fields)} ~'k)
                                             (dissoc (with-meta (into {} ~'this) ~'__meta) ~'k)
                                             (new ~tagname ~@(remove #{'__extmap} fields) 
-                                                 (not-empty (dissoc ~'__extmap ~'k))))))])]
-     (let [[i m] (-> [interfaces methods] eqhash iobj ilookup imap)]
+                                                 (not-empty (dissoc ~'__extmap ~'k))))))])
+      (ijavamap [[i m]]
+                [(conj i 'java.util.Map)
+                 (conj m
+                       `(size [~'this] (.count ~'this))
+                       `(isEmpty [~'this] (= 0 (.count ~'this)))
+                       `(containsValue [~'this ~'v] (-> ~'this vals (.contains ~'v)))
+                       `(get [~'this ~'k] (.valAt ~'this ~'k))
+                       `(put [~'this ~'k ~'v] (throw (UnsupportedOperationException.)))
+                       `(remove [~'this ~'k] (throw (UnsupportedOperationException.)))
+                       `(putAll [~'this ~'m] (throw (UnsupportedOperationException.)))
+                       `(clear [~'this] (throw (UnsupportedOperationException.)))
+                       `(keySet [~'this] (set (keys ~'this)))
+                       `(values [~'this] (vals ~'this))
+                       `(entrySet [~'this] (set ~'this)))])
+      ]
+     (let [[i m] (-> [interfaces methods] eqhash iobj ilookup imap ijavamap)]
        `(deftype* ~tagname ~classname ~(conj hinted-fields '__meta '__extmap) 
           :implements ~(vec i) 
           ~@m)))))

test/clojure/test_clojure/protocols.clj

@@ -30,6 +30,7 @@
      (map #(.getName %))
      (sort)))
 
+(defrecord EmptyRecord [])
 (defrecord TestRecord [a b])
 (defn r
   ([a b] (TestRecord. a b))
@@ -157,6 +158,26 @@
     (is (= (r 1 3 {} {:c 4}) (merge rec {:b 3 :c 4})))))
 
 (deftest defrecord-interfaces-test
+  (testing "java.util.Map"
+    (let [rec (r 1 2)]
+      (is (= 2 (.size rec)))
+      (is (= 3 (.size (assoc rec :c 3))))
+      (is (not (.isEmpty rec)))
+      (is (.isEmpty (EmptyRecord.)))
+      (is (.containsKey rec :a))
+      (is (not (.containsKey rec :c)))
+      (is (.containsValue rec 1))
+      (is (not (.containsValue rec 3)))
+      (is (= 1 (.get rec :a)))
+      (is (thrown? UnsupportedOperationException (.put rec :a 1)))
+      (is (thrown? UnsupportedOperationException (.remove rec :a)))
+      (is (thrown? UnsupportedOperationException (.putAll rec {})))
+      (is (thrown? UnsupportedOperationException (.clear rec)))
+      (is (= #{:a :b} (.keySet rec)))
+      (is (= #{1 2} (set (.values rec))))
+      (is (= #{[:a 1] [:b 2]} (.entrySet rec)))
+      
+      ))
   (testing "IPersistentCollection"
     (testing ".cons"
       (let [rec (r 1 2)]