Clojure

clojure.core/bean should implement Iterable

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: Release 1.7
  • Component/s: None
  • Labels:
  • Patch:
    Code and Test
  • Approval:
    Ok

Description

The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator;  clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)

Patch adds `iterator` method to clojure.core/bean.

Screened by: Alex Miller

Activity

Ambrose Bonnaire-Sergeant made changes -
Field Original Value New Value
Description The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterable` method in APersistentMap copied from PersistentHashMap.
The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method in APersistentMap copied from PersistentHashMap.
Ambrose Bonnaire-Sergeant made changes -
Description The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method in APersistentMap copied from PersistentHashMap.
The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method in APersistentMap copied from PersistentHashMap.
Alex Miller made changes -
Approval Triaged [ 10120 ]
Ambrose Bonnaire-Sergeant made changes -
Description The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method in APersistentMap copied from PersistentHashMap.
The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method to clojure.core/bean.
Attachment iterable-bean-v2.diff [ 13132 ]
Ambrose Bonnaire-Sergeant made changes -
Attachment iterable-bean.diff [ 13131 ]
Andy Fingerhut made changes -
Patch Code and Test [ 10002 ]
Alex Miller made changes -
Labels ft interop
Alex Miller made changes -
Description The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method to clojure.core/bean.
The changes in Clojure 1.6 hashing revealed that `bean` does not return a map that implements Iterable:

{code}
user=> (hash (bean (java.util.Date.)))

AbstractMethodError clojure.lang.APersistentMap.iterator()Ljava/util/Iterator; clojure.core.proxy$clojure.lang.APersistentMap$ff19274a.iterator (:-1)
{code}

Patch adds `iterator` method to clojure.core/bean.

*Screened by:* Alex Miller
Alex Miller made changes -
Priority Minor [ 4 ] Major [ 3 ]
Rich Hickey made changes -
Approval Triaged [ 10120 ] Ok [ 10007 ]
Fix Version/s Release 1.7 [ 10250 ]
Stuart Halloway made changes -
Resolution Completed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]

People

Vote (2)
Watch (4)

Dates

  • Created:
    Updated:
    Resolved: