I've done a tiny improvement to this implementation. Please Xavier review it ;). http://github.com/spastorino/rails/commit/63b43575765c7d4042c23ac3ddb1213c1968b5a3
On Sun, Jun 13, 2010 at 1:42 AM, Xavier Noria <[email protected]> wrote: > On Sun, Jun 13, 2010 at 4:42 AM, Xavier Noria <[email protected]> wrote: > >> Just realized the patch is not compatible with 1.9, since in 1.9 only >> existing keys are merged (that's the key? key test in Rubinius). Will >> revise it myself. > > Rather, yield is called only for existing keys. This revisions seems > to be correct: > > http://github.com/rails/rails/commit/36143d26cb841210b5f22aff4ed9c093a0554a1a > > -- > You received this message because you are subscribed to the Google Groups > "Ruby on Rails: Core" group. > To post to this group, send email to [email protected]. > To unsubscribe from this group, send email to > [email protected]. > For more options, visit this group at > http://groups.google.com/group/rubyonrails-core?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en.
