Project

General

Profile

Actions

Feature #8458

closed

Array#merge/merge!

Added by Anonymous almost 11 years ago. Updated almost 11 years ago.

Status:
Rejected
Assignee:
-
Target version:
-
[ruby-core:55186]

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?

Updated by matz (Yukihiro Matsumoto) almost 11 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.

Updated by Anonymous almost 11 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.

Actions

Also available in: Atom PDF

Like0
Like0Like0