From: Ethan - <redmine@...>
Date: 2010-10-13T03:21:27+09:00
Subject: [ruby-core:32759] [Ruby 1.8-Bug#3938][Open] Ruby incorrectly compares the length of Array elements of Enumerable objects to the arity of Methods given as blocks to Enumerable methods


--mimepart_4cb4a6a23eec9_ef0d9f371c189c9
Content-Type: text/plain
Content-Transfer-Encoding: Quoted-printable
Content-Disposition: inline

Bug #3938: Ruby incorrectly compares the length of Array elements of Enum=
erable objects to the arity of Methods given as blocks to Enumerable meth=
ods
http://redmine.ruby-lang.org/issues/show/3938

Author: Ethan -
Status: Open, Priority: Normal
Category: core
ruby -v: ruby 1.8.6 (2009-06-08 patchlevel 369) [universal-darwin9.0] (an=
d others, see description)

Given an Enumerable object that has an element which is an Array, when an=
 Enumerable method which takes a block (#map, #detect, etc) is called on =
that object and given a Method as its block, Ruby will incorrectly compar=
e the length of the Array to the arity of the Method. =


  def Object.onearg(arg)
    arg
  end
  amethod =3D Object.method(:onearg)
  aproc =3D proc{|arg| arg}

amethod and aproc both have arity of 1. they should generally behave the =
same. but, they behave differently when given as a block to Enumerable me=
thods, with amethod behaving incorrectly when the Enumerable in question =
contains an array: =


  >> [[1, 2]].detect(&amethod)
  ArgumentError: wrong number of arguments (2 for 1)

this seems to incorrectly compare the length of the element [1, 2] to the=
 arity of amethod, even though it's passing one argument (the array [1, 2=
]) to amethod. =


the Proc behaves correctly:

  >> [[1, 2]].detect(&aproc)
  =3D> [1, 2]

this does not compare the element's length to the arity of aproc, and so =
works fine. =


Giving the Method as a block works fine when the arity happens to be the =
same as the length of the element which is an array:

  > [[1]].detect(&amethod)
  =3D> [1]

Even though it is passed the whole array (seen in the return value), and =
not the element of the array. =


File is attached to minimally reproduce, and its output is:

[1, 2]
[1]
methodarity.rb:10:in `onearg': wrong number of arguments (2 for 1) (Argum=
entError)
	from methodarity.rb:10:in `to_proc'
	from methodarity.rb:7:in `detect'
	from methodarity.rb:10:in `each'
	from methodarity.rb:10:in `detect'
	from methodarity.rb:10


Tested on:
ruby 1.8.6 (2009-06-08 patchlevel 369) [universal-darwin9.0]
ruby 1.8.6 (2010-02-05 patchlevel 399) [i686-darwin9.8.0] =

ruby 1.8.7 (2009-06-12 patchlevel 174) [i686-darwin9.8.0]
ruby 1.8.7 (2010-06-23 patchlevel 299) [i686-darwin9.8.0] =

ruby 1.8.6 (2007-09-24 patchlevel 111) [i386-mswin32]
ruby 1.8.6 (2010-02-04 patchlevel 398) [i386-mingw32]
ruby 1.8.7 (2010-01-10 patchlevel 249) [i386-mingw32]

Behaves correctly in ruby 1.9.*.


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

--mimepart_4cb4a6a23eec9_ef0d9f371c189c9
Content-Type: text/x-ruby-script; name=methodarity.rb
Content-Transfer-Encoding: Base64
Content-Disposition: attachment; filename=methodarity.rb

ZGVmIE9iamVjdC5vbmVhcmcoYXJnKQogIGFyZwplbmQKYW1ldGhvZCA9IE9i
amVjdC5tZXRob2QoOm9uZWFyZykKYXByb2MgPSBwcm9je3xhcmd8IGFyZ30K
CnJlcXVpcmUgJ3BwJwpwcChbWzEsIDJdXS5kZXRlY3QoJmFwcm9jKSkgICAj
IHdvcmtzIGJlY2F1c2UgcHJvYyB3b3JrcyBjb3JyZWN0bHkKcHAoW1sxXV0u
ZGV0ZWN0KCZhbWV0aG9kKSkgICAgIyB3b3JrcyBiZWNhdXNlIGFyaXR5IGhh
cHBlbnMgdG8gYmUgdGhlIHNhbWUgYXMgWzFdLnNpemUKcHAoW1sxLCAyXV0u
ZGV0ZWN0KCZhbWV0aG9kKSkgIyBkb2Vzbid0IHdvcmsgYmVjYXVzZSBhcml0
eSBpc24ndCB0aGUgYXNtZSBhcyBbMSwgMl0uc2l6ZQo=

--mimepart_4cb4a6a23eec9_ef0d9f371c189c9--