Clojure

Use transient map in zipmap

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: Release 1.5
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code
  • Approval:
    Vetted

Description

The attached patch changes zipmap to use a transient map internally. The definition is also moved so that it resides below that of #'transient. The original definition is commented out (like that of #'into).

Activity

Rich Hickey made changes -
Field Original Value New Value
Approval Vetted [ 10003 ]
Fix Version/s Release 1.5 [ 10150 ]
Aaron Bedra made changes -
Approval Vetted [ 10003 ] Incomplete [ 10006 ]
Michał Marczyk made changes -
Attachment 0001-Use-transient-map-in-zipmap.patch [ 11428 ]
Michał Marczyk made changes -
Attachment 0001-Use-transient-map-in-zipmap.patch [ 11428 ]
Michał Marczyk made changes -
Andy Fingerhut made changes -
Approval Incomplete [ 10006 ]
Stuart Halloway made changes -
Fix Version/s Release 1.5 [ 10150 ]
Fix Version/s Release 1.6 [ 10157 ]
Aaron Bedra made changes -
Comment [ Is there a reason why the function moved inside of core.clj? ]
Aaron Bedra made changes -
Comment [ Never mind. I just took a look and realized that transient wasn't available yet. Sorry. ]
Aaron Bedra made changes -
Assignee Michał Marczyk [ michalmarczyk ] Aaron Bedra [ aaron ]
Aaron Bedra made changes -
Affects Version/s Release 1.6 [ 10157 ]
Alex Miller made changes -
Approval Triaged [ 10120 ]
Affects Version/s Release 1.6 [ 10157 ]
Affects Version/s Release 1.5 [ 10150 ]
Fix Version/s Release 1.6 [ 10157 ]
Alex Miller made changes -
Labels performance
Rich Hickey made changes -
Approval Triaged [ 10120 ] Vetted [ 10003 ]
Michał Marczyk made changes -

People

Vote (5)
Watch (5)

Dates

  • Created:
    Updated: