diff options
author | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2003-07-21 18:48:58 +0000 |
---|---|---|
committer | matz <matz@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2003-07-21 18:48:58 +0000 |
commit | c26663f24b0c26906b62d70ae113fbc45efe29f8 (patch) | |
tree | 6edefae6c56ef50b55a4ef0795f29adf35e181a6 | |
parent | fa284f8127be2cccb3775309fd5747d79f55ca3d (diff) |
* numeric.c (num_equal): should not use rb_equal().
* object.c (rb_equal): back out the last change.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@4113 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | numeric.c | 6 | ||||
-rw-r--r-- | object.c | 2 |
3 files changed, 7 insertions, 5 deletions
@@ -1,10 +1,10 @@ Tue Jul 22 02:22:45 2003 Yukihiro Matsumoto <matz@ruby-lang.org> + * numeric.c (num_equal): should not use rb_equal(). + * string.c (rb_str_equal): should return nil for non string operand to conform comparable convention. [ruby-dev:20759] - * object.c (rb_equal): preserve nil if "==" returns nil. - Tue Jul 22 00:19:19 2003 Yukihiro Matsumoto <matz@ruby-lang.org> * lib/tmpdir.rb: new library to get temporary directory path, @@ -57,7 +57,7 @@ #define DBL_EPSILON 2.2204460492503131e-16 #endif -static ID id_coerce, id_to_i; +static ID id_coerce, id_to_i, id_eq; VALUE rb_cNumeric; VALUE rb_cFloat; @@ -515,7 +515,8 @@ static VALUE num_equal(x, y) VALUE x, y; { - return rb_equal(y, x); + if (x == y) return Qtrue; + return rb_funcall(y, id_eq, 1, x); } static VALUE @@ -1804,6 +1805,7 @@ Init_Numeric() #endif id_coerce = rb_intern("coerce"); id_to_i = rb_intern("to_i"); + id_eq = rb_intern("=="); rb_eZeroDivError = rb_define_class("ZeroDivisionError", rb_eStandardError); rb_eFloatDomainError = rb_define_class("FloatDomainError", rb_eRangeError); @@ -42,7 +42,7 @@ rb_equal(obj1, obj2) if (obj1 == obj2) return Qtrue; result = rb_funcall(obj1, id_eq, 1, obj2); if (RTEST(result)) return Qtrue; - return result; + return Qfalse; } int |