diff --git a/ChangeLog b/ChangeLog index f04f4ff1d7..1bc049031c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,4 +1,7 @@ -Fri Nov 30 17:43:39 2012 Nobuyoshi Nakada +Fri Nov 30 17:43:45 2012 Nobuyoshi Nakada + + * string.c (rb_str_cmp_m): use rb_check_funcall instead of respond_to + and call. * string.c (rb_str_cmp_m): return fixed value, one of -1,0,+1 always. diff --git a/string.c b/string.c index 4270460e16..c6379bbf5b 100644 --- a/string.c +++ b/string.c @@ -2385,15 +2385,15 @@ rb_str_cmp_m(VALUE str1, VALUE str2) int result; if (!RB_TYPE_P(str2, T_STRING)) { + VALUE tmp; if (!rb_respond_to(str2, rb_intern("to_str"))) { return Qnil; } - else if (!rb_respond_to(str2, rb_intern("<=>"))) { + else if ((tmp = rb_check_funcall(str2, rb_intern("<=>"), 1, &str1)) == + Qundef) { return Qnil; } else { - VALUE tmp = rb_funcall(str2, rb_intern("<=>"), 1, str1); - if (NIL_P(tmp)) return Qnil; result = -rb_cmpint(tmp, str1, str2); }