http://dev.clojure.org/jira This file is an XML representation of an issue en-us 4.4 649 25-07-2011 http://dev.clojure.org/jira/browse/CLJ-370 Clojure <p>non-reflective dynamic creation of defrecord instances via a multimethod</p> CLJ-370

record multimethod

Enhancement Major Closed Declined Unassigned Assembla Importer Tue, 1 Jun 2010 20:06:00 -0500 Sun, 16 Oct 2011 05:33:31 -0500 Sun, 16 Oct 2011 05:33:30 -0500 0 1 <p>Converted from <a href="http://www.assembla.com/spaces/clojure/tickets/370">http://www.assembla.com/spaces/clojure/tickets/370</a><br/> Attachments:<br/> 0370-record-multimethod.patch - <a href="https://www.assembla.com/spaces/clojure/documents/d2J0riBEKr35rdeJe5cbLA/download/d2J0riBEKr35rdeJe5cbLA">https://www.assembla.com/spaces/clojure/documents/d2J0riBEKr35rdeJe5cbLA/download/d2J0riBEKr35rdeJe5cbLA</a><br/> 0370-record-multimethod-take-2.patch - <a href="https://www.assembla.com/spaces/clojure/documents/aGvvlSCmWr35fseJe5cbCb/download/aGvvlSCmWr35fseJe5cbCb">https://www.assembla.com/spaces/clojure/documents/aGvvlSCmWr35fseJe5cbCb/download/aGvvlSCmWr35fseJe5cbCb</a></p> <p>stu said: [<a href="file:d2J0riBEKr35rdeJe5cbLA">file:d2J0riBEKr35rdeJe5cbLA</a>]</p> <p>stu said: Questions:</p> <ul> <li>I don't like that create-record is public in core. Change its name? Throw in another namespace?</li> </ul> <p>richhickey said: how about record-implementation-detail-multimethod?</p> <p>Inside emit-record-factory-method, how about:</p> <ul class="alternate" type="square"> <li>using :keys destructuring for fields</li> <li>no need for apply on dissoc - it takes multiple keys</li> </ul> <p>Also, you might want to make record itself a macro, so you can flow metadata to the created record:</p> <p>^{:my "meta"}(record ::Foo :a 1 :b 2)</p> <p>This will be useful since this will likely become at least the print-dup form for records</p> <p>stu said: I was thinking that record might often get apply-ed. Do we need a function and a macro?</p> <p>stu said: [<a href="file:aGvvlSCmWr35fseJe5cbCb">file:aGvvlSCmWr35fseJe5cbCb</a>]</p> <p>stu said: second patch subsumes first and incorporates Rich's feedback. print/read moved to separate ticket #374</p> <p>richhickey said: second thoughts here</p> <p>stu said: Updating tickets (#370, #366, #374)</p> <p>Rich: updated design proposal at <a href="http://dev.clojure.org/display/design/defrecord+improvements">http://dev.clojure.org/display/design/defrecord+improvements</a>. Please provide feedback there and then mark this as waiting on me.</p> <p>Old ticket. Goal was accomplished via factory fns and literals.</p> Approval Vetted Global Rank Waiting On stu