diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-12-04 07:37:10 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2009-12-04 07:37:10 +0000 |
commit | 2c2cf71b048e42950d5d5f79fe78bc9fea0df158 (patch) | |
tree | 55d50b0856f96e2a2db8d68d8555403078531a61 | |
parent | 4cec4edd0bf2361a1769d108c910c4710e3c290b (diff) | |
download | ruby-2c2cf71b048e42950d5d5f79fe78bc9fea0df158.tar.gz |
* time.c (num_exact): should not accept strings as operands, even
though they respond to #to_r. ideally, strict rational
conversion should be done by a method like #to_rational, not #to_r.
[ruby-core:23729]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@25991 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | time.c | 7 |
2 files changed, 12 insertions, 2 deletions
@@ -3,6 +3,13 @@ Fri Dec 4 15:50:18 2009 Shugo Maeda <shugo@ruby-lang.org> * vm_eval.c (yield_under): does not yield self, and passes blockptr instead of &block to vm_cref_push(). [ruby-dev:39833] +Fri Dec 4 15:15:43 2009 Yukihiro Matsumoto <matz@ruby-lang.org> + + * time.c (num_exact): should not accept strings as operands, even + though they respond to #to_r. ideally, strict rational + conversion should be done by a method like #to_rational, not #to_r. + [ruby-core:23729] + Fri Dec 4 13:10:23 2009 NAKAMURA, Hiroshi <nahi@ruby-lang.org> * ext/stringio/stringio.c (ungetc): RDoc updated. trunk allows @@ -214,19 +214,22 @@ num_exact(VALUE v) v = rb_convert_type(v, T_RATIONAL, "Rational", "to_r"); break; + case T_STRING: case T_NIL: goto typeerror; default: { VALUE tmp; - if (!NIL_P(tmp = rb_check_convert_type(v, T_RATIONAL, "Rational", "to_r"))) + if (!NIL_P(tmp = rb_check_convert_type(v, T_RATIONAL, "Rational", "to_r"))) { + if (rb_respond_to(v, rb_intern("to_str"))) goto typeerror; v = tmp; + } else if (!NIL_P(tmp = rb_check_to_integer(v, "to_int"))) v = tmp; else { typeerror: rb_raise(rb_eTypeError, "can't convert %s into an exact number", - rb_obj_classname(v)); + NIL_P(v) ? "nil" : rb_obj_classname(v)); } break; } |