diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-01-10 13:41:18 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2017-01-10 13:41:18 +0000 |
commit | a3fb17f3a0fbc960d224cb208fb705065829241c (patch) | |
tree | 94fc73fc466cab366907424dfe5883c2470e55ea /rational.c | |
parent | 14b3dc1ec45861724fcfa1ae44fabe2cc7f03bf9 (diff) | |
download | ruby-a3fb17f3a0fbc960d224cb208fb705065829241c.tar.gz |
rational.c: short circuit optimization
* rational.c (nurat_reduce): short circuit when arguments are ONE,
nothing is needed.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@57299 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'rational.c')
-rw-r--r-- | rational.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/rational.c b/rational.c index 212ad07faa..26324899e5 100644 --- a/rational.c +++ b/rational.c @@ -488,7 +488,9 @@ nurat_canonicalize(VALUE *num, VALUE *den) static void nurat_reduce(VALUE *x, VALUE *y) { - VALUE gcd = f_gcd(*x, *y); + VALUE gcd; + if (*x == ONE || *y == ONE) return; + gcd = f_gcd(*x, *y); *x = f_idiv(*x, gcd); *y = f_idiv(*y, gcd); } |