TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Remove blockless_step leave all that up to Ruby

Browse Source 
Closes #6297
This commit is contained in:
Santiago Pastorino 2012-05-13 13:44:35 -03:00
parent 6e55d30ca8
commit 1c481be331
3 changed files with 1 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
activesupport/lib/active_support/core_ext/range.rb
View File

@ -1,4 +1,3 @@
require 'active_support/core_ext/range/blockless_step'
require 'active_support/core_ext/range/conversions'
require 'active_support/core_ext/range/include_range'
require 'active_support/core_ext/range/overlaps'

13
activesupport/lib/active_support/core_ext/range/blockless_step.rb
View File

@ -1,13 +0,0 @@
require 'active_support/core_ext/module/aliasing'
class Range
def step_with_blockless(*args, &block) #:nodoc:
if block_given?
step_without_blockless(*args, &block)
else
step_without_blockless(*args).to_a
end
end
alias_method_chain :step, :blockless
end

2
activesupport/test/core_ext/range_ext_test.rb
View File

@ -70,7 +70,7 @@ def test_should_include_identical_exclusive_with_floats
end
def test_blockless_step
assert_equal [1,3,5,7,9], (1..10).step(2)
assert_equal [1,3,5,7,9], (1..10).step(2).to_a
end
def test_original_step