diff --git a/numeric.c b/numeric.c index 2784b6dd86..b34fb0a5f9 100644 --- a/numeric.c +++ b/numeric.c @@ -5158,6 +5158,7 @@ static VALUE rb_int_s_isqrt(VALUE self, VALUE num) { unsigned long n, sq; + num = rb_to_int(num); if (FIXNUM_P(num)) { if (FIXNUM_NEGATIVE_P(num)) { domain_error("isqrt"); @@ -5166,7 +5167,7 @@ rb_int_s_isqrt(VALUE self, VALUE num) sq = rb_ulong_isqrt(n); return LONG2FIX(sq); } - if (RB_TYPE_P(num, T_BIGNUM)) { + else { size_t biglen; if (RBIGNUM_NEGATIVE_P(num)) { domain_error("isqrt"); @@ -5183,7 +5184,6 @@ rb_int_s_isqrt(VALUE self, VALUE num) #endif return rb_big_isqrt(num); } - return Qnil; } /* diff --git a/test/ruby/test_integer.rb b/test/ruby/test_integer.rb index 05b0b68ccc..e0927c9ffe 100644 --- a/test/ruby/test_integer.rb +++ b/test/ruby/test_integer.rb @@ -466,6 +466,7 @@ class TestInteger < Test::Unit::TestCase end def test_square_root + assert_raise(TypeError) {Integer.sqrt("x")} assert_raise(Math::DomainError) {Integer.sqrt(-1)} assert_equal(0, Integer.sqrt(0)) (1...4).each {|i| assert_equal(1, Integer.sqrt(i))}