Feature #8458

Array#merge/merge!

Added by Boris Stitnicky about 2 years ago. Updated about 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 about 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 Boris Stitnicky about 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