diff options
author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-06-30 16:03:41 +0000 |
---|---|---|
committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-06-30 16:03:41 +0000 |
commit | 5f8e0b66330ece522adb41c1e5d09babdb0cf7d2 (patch) | |
tree | cbd9b63dab6515ad2b364606cb2e465ff221e48a | |
parent | 4c4357c6986e80d7b78d2cdb103e827a89373090 (diff) |
* numeric.c (num_coerce): call rb_Float(x) first. don't depend on
evaluation order of function arguments.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@17750 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | numeric.c | 4 |
2 files changed, 8 insertions, 1 deletions
@@ -1,3 +1,8 @@ +Tue Jul 1 00:59:43 2008 Tanaka Akira <akr@fsij.org> + + * numeric.c (num_coerce): call rb_Float(x) first. don't depend on + evaluation order of function arguments. + Tue Jul 1 00:49:11 2008 Nobuyoshi Nakada <nobu@ruby-lang.org> * gc.c (rb_newobj): abort GC phase before rb_bug. @@ -118,7 +118,9 @@ num_coerce(VALUE x, VALUE y) { if (CLASS_OF(x) == CLASS_OF(y)) return rb_assoc_new(y, x); - return rb_assoc_new(rb_Float(y), rb_Float(x)); + x = rb_Float(x); + y = rb_Float(y); + return rb_assoc_new(y, x); } static VALUE |