Feature #8458

Array#merge/merge!

Added by Anonymous over 2 years ago. Updated over 2 years ago.

[ruby-core:55186]
Status:Rejected
Priority:Normal
Assignee:-

Description

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?

History

#1 Updated by Yukihiro Matsumoto over 2 years ago

  • Status changed from Open to Rejected

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.

#2 Updated by Anonymous over 2 years ago

Still, many thanks for [:A, :B, C: 1] syntax, 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.

Also available in: Atom PDF