Converted from http://www.assembla.com/spaces/clojure/tickets/370
Attachments:
0370-record-multimethod.patch - https://www.assembla.com/spaces/clojure/documents/d2J0riBEKr35rdeJe5cbLA/download/d2J0riBEKr35rdeJe5cbLA
0370-record-multimethod-take-2.patch - https://www.assembla.com/spaces/clojure/documents/aGvvlSCmWr35fseJe5cbCb/download/aGvvlSCmWr35fseJe5cbCb

stu said: [file:d2J0riBEKr35rdeJe5cbLA]

stu said: Questions:

• I don't like that create-record is public in core. Change its name? Throw in another namespace?

richhickey said: how about record-implementation-detail-multimethod?

Inside emit-record-factory-method, how about:

• using :keys destructuring for fields
• no need for apply on dissoc - it takes multiple keys

Also, you might want to make record itself a macro, so you can flow metadata to the created record:

^{:my "meta"}(record ::Foo :a 1 :b 2)

This will be useful since this will likely become at least the print-dup form for records

stu said: I was thinking that record might often get apply-ed. Do we need a function and a macro?

stu said: [file:aGvvlSCmWr35fseJe5cbCb]

stu said: second patch subsumes first and incorporates Rich's feedback. print/read moved to separate ticket #374

richhickey said: second thoughts here

stu said: Updating tickets (#370, #366, #374)

Rich: updated design proposal at http://dev.clojure.org/display/design/defrecord+improvements. Please provide feedback there and then mark this as waiting on me.

Old ticket. Goal was accomplished via factory fns and literals.