Clojure

clojure.core/merge-with doesn't work with defrecord types

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: Release 1.2
  • Component/s: None
  • Labels:
    None
  • Approval:
    Ok

Description

clojure.core/merge-with works only with map types that implement clojure.lang.IFn for element lookup. In particular, it doesn't work with types defined by defrecord.

http://groups.google.com/group/clojure-dev/browse_thread/thread/8e546e595a4c4cab/aa47af1770183c56?lnk=gst&q=merge-with#aa47af1770183c56

Activity

Hide
Assembla Importer added a comment -

stu said: Konrad,

Can you please add a pointer to the discussion on the mailing list or irc that led to a ticket being created? It doesn't seem like much, but it is as huge help for me having context when I have a bunch of patches to review.

Thanks!
Stu

Show
Assembla Importer added a comment - stu said: Konrad, Can you please add a pointer to the discussion on the mailing list or irc that led to a ticket being created? It doesn't seem like much, but it is as huge help for me having context when I have a bunch of patches to review. Thanks! Stu
Hide
Assembla Importer added a comment -
Show
Assembla Importer added a comment - stu said: [file:akEK3iBO4r356HeJe5cbLr]
Hide
Assembla Importer added a comment -

stu said: second patch subsumes first and adds test

Show
Assembla Importer added a comment - stu said: second patch subsumes first and adds test
Hide
Assembla Importer added a comment -

khinsen said: Stu,

I just added the pointer to the description!

Konrad.

Show
Assembla Importer added a comment - khinsen said: Stu, I just added the pointer to the description! Konrad.
Hide
Assembla Importer added a comment -

stu said: Updating tickets (#327, #359, #360, #361, #362)

Show
Assembla Importer added a comment - stu said: Updating tickets (#327, #359, #360, #361, #362)

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: