From: "boris_stitnicky (Boris Stitnicky)" <boris@...>
Date: 2013-05-30T01:59:06+09:00
Subject: [ruby-core:55201] [ruby-trunk - Feature #8458] Array#merge/merge!


Issue #8458 has been updated by boris_stitnicky (Boris Stitnicky).


Still, many thanks for [:A, :B, C: 1], I was very happy to discover it!

matz (Yukihiro Matsumoto) wrote:
> Generally speaking, arrays are not argument lists.  This method is specific to argument lists.
> Thus Array is not the right place for the method.
> 
> Matz.


----------------------------------------
Feature #8458: Array#merge/merge!
https://bugs.ruby-lang.org/issues/8458#change-39561

Author: boris_stitnicky (Boris Stitnicky)
Status: Rejected
Priority: Normal
Assignee: 
Category: 
Target version: 


Very simple method can be defined:

class Array
  def merge! other
    require 'active_support/all' # can't use #tap, see #8457
    tail = extract_options!
    other_tail = other.extract_options!
    concat( other ) << tail.update( other_tail )
  end
end

When pondering about where this method should dwell, it seemed to me, perhaps in the core itself? Am I deluding myself?


-- 
http://bugs.ruby-lang.org/