diff options
author | Nobuyoshi Nakada <nobu@ruby-lang.org> | 2021-01-23 10:28:07 +0900 |
---|---|---|
committer | Nobuyoshi Nakada <nobu@ruby-lang.org> | 2021-01-23 10:30:59 +0900 |
commit | a55eb9a2af7950d180d9d31ffde2bce66710f44f (patch) | |
tree | 4b9863c2aef76c20cd7c7ea7d6450d33c2915234 | |
parent | 37258b64c759c0add4d97f79aeeda8ae4ffc3677 (diff) | |
download | ruby-a55eb9a2af7950d180d9d31ffde2bce66710f44f.tar.gz |
Make reciprocal properly of non-integral rational [Bug #17572]
-rw-r--r-- | rational.c | 2 | ||||
-rw-r--r-- | test/ruby/test_rational.rb | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/rational.c b/rational.c index 4a7df253a5..8548b3bd43 100644 --- a/rational.c +++ b/rational.c @@ -1868,7 +1868,7 @@ VALUE rb_rational_reciprocal(VALUE x) { get_dat1(x); - return f_rational_new_no_reduce2(CLASS_OF(x), dat->den, dat->num); + return nurat_convert(CLASS_OF(x), dat->den, dat->num, FALSE); } /* diff --git a/test/ruby/test_rational.rb b/test/ruby/test_rational.rb index 820dd30a82..fe9de64c4c 100644 --- a/test/ruby/test_rational.rb +++ b/test/ruby/test_rational.rb @@ -613,6 +613,13 @@ class Rational_Test < Test::Unit::TestCase assert_nothing_raised(TypeError, '[Bug #5020] [ruby-dev:44088]') do Rational(1,2).coerce(Complex(1,1)) end + + assert_raise(ZeroDivisionError) do + 1 / 0r.coerce(0+0i)[0] + end + assert_raise(ZeroDivisionError) do + 1 / 0r.coerce(0.0+0i)[0] + end end class ObjectX |