Skip to content

Commit

Permalink
Merge pull request #3293 from ksss/enum-first
Browse files Browse the repository at this point in the history
Fix incompatibility for Enumerable#first
  • Loading branch information
matz committed Nov 26, 2016
2 parents f666350 + 1bf565e commit 4256130
Showing 1 changed file with 16 additions and 9 deletions.
25 changes: 16 additions & 9 deletions mrbgems/mruby-enum-ext/mrblib/enum.rb
Expand Up @@ -215,21 +215,28 @@ def sort_by(&block)
# Returns the first element, or the first +n+ elements, of the enumerable.
# If the enumerable is empty, the first form returns <code>nil</code>, and the
# second form returns an empty array.
def first(n=NONE)
if n == NONE
def first(*args)
case args.length
when 0
self.each do |*val|
return val.__svalue
end
return nil
else
a = []
i = 0
when 1
n = args[0]
raise TypeError, "no implicit conversion of #{n.class} into Integer" unless n.respond_to?(:to_int)
i = n.to_int
raise ArgumentError, "attempt to take negative size" if i < 0
ary = []
return ary if i == 0
self.each do |*val|
break if n<=i
a.push val.__svalue
i += 1
ary << val.__svalue
i -= 1
break if i == 0
end
a
ary
else
raise ArgumentError, "wrong number of arguments (given #{args.length}, expected 0..1)"
end
end

Expand Down

0 comments on commit 4256130

Please sign in to comment.