summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormrkn <mrkn@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-09-12 05:35:46 +0000
committermrkn <mrkn@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2018-09-12 05:35:46 +0000
commit0a0f8238d02a2dfff4cd2892408e14cb826cec7e (patch)
tree8904010c4daedbbdc0d50480d5e652e63f426df3
parent0f6708eb9b0aa0826740656d52f6c65b32c2ec77 (diff)
enumerator.c: Fix ArithmeticSequence for complex step
Make sure Enumerator::ArithmeticSequence#each to work well for a complex step value. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64692 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r--enumerator.c3
-rw-r--r--test/ruby/test_arithmetic_sequence.rb20
2 files changed, 22 insertions, 1 deletions
diff --git a/enumerator.c b/enumerator.c
index 2c115aa740..46ded3760e 100644
--- a/enumerator.c
+++ b/enumerator.c
@@ -2661,7 +2661,8 @@ arith_seq_each(VALUE self)
s = arith_seq_step(self);
x = arith_seq_exclude_end_p(self);
- if (ruby_float_step(c, e, s, x, TRUE)) {
+ if (RB_TYPE_P(c, T_FLOAT) && RB_TYPE_P(s, T_FLOAT) &&
+ ruby_float_step(c, e, s, x, TRUE)) {
return self;
}
diff --git a/test/ruby/test_arithmetic_sequence.rb b/test/ruby/test_arithmetic_sequence.rb
index a9978733fa..f4f47ea7a8 100644
--- a/test/ruby/test_arithmetic_sequence.rb
+++ b/test/ruby/test_arithmetic_sequence.rb
@@ -396,4 +396,24 @@ class TestArithmeticSequence < Test::Unit::TestCase
assert_equal(4, (10...2).step(-2).size)
assert_equal(Float::INFINITY, (1..).step(-1).size)
end
+
+ def assert_num_equal_type(ary1, ary2, message=nil)
+ assert_equal(ary1.length, ary2.length, message)
+ ary1.zip(ary2) do |e1, e2|
+ assert_equal(e1.class, e2.class, message)
+ if e1.is_a? Complex
+ assert_equal(e1.real, e2.real, message)
+ assert_equal(e1.imag, e2.imag, message)
+ else
+ assert_equal(e1, e2, message)
+ end
+ end
+ end
+
+ def test_complex
+ assert_num_equal_type([1, 1+1i, 1+2i], (1..).step(1i).take(3))
+ assert_num_equal_type([1, 1+1.0i, 1+2.0i], (1..).step(1.0i).take(3))
+ assert_num_equal_type([0.0, 0.0+1.0i, 0.0+2.0i], (0.0..).step(1.0i).take(3))
+ assert_num_equal_type([0.0+0.0i, 0.0+1.0i, 0.0+2.0i], (0.0i..).step(1.0i).take(3))
+ end
end