mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
* compar.c (cmp_equal): warn for this release and still rescue
standard exceptions for a nicer transition. See #7688. Partly reverts r44502. * test/ruby/test_comparable.rb: adapt assertion to match new behavior. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@44646 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
35bf7276d9
commit
d1d7f12c89
3 changed files with 39 additions and 8 deletions
13
ChangeLog
13
ChangeLog
|
@ -1,3 +1,16 @@
|
|||
Sun Jan 19 06:38:48 2014 Benoit Daloze <eregontp@gmail.com>
|
||||
|
||||
* compar.c (cmp_equal): warn for this release and still rescue
|
||||
standard exceptions for a nicer transition. See #7688.
|
||||
Partly reverts r44502.
|
||||
|
||||
* test/ruby/test_comparable.rb: adapt assertion to match new behavior.
|
||||
|
||||
Sun Jan 19 06:27:18 2014 Benoit Daloze <eregontp@gmail.com>
|
||||
|
||||
* test/ruby/test_comparable.rb: specify behavior for the different
|
||||
kind of exceptions rescued (or not) by Comparable#==.
|
||||
|
||||
Sat Jan 18 23:12:19 2014 Tanaka Akira <akr@fsij.org>
|
||||
|
||||
* ext/socket: Avoid unnecessary ppoll/select on Linux.
|
||||
|
|
28
compar.c
28
compar.c
|
@ -58,6 +58,24 @@ cmp_eq_recursive(VALUE arg1, VALUE arg2, int recursive)
|
|||
return rb_funcallv(arg1, cmp, 1, &arg2);
|
||||
}
|
||||
|
||||
static VALUE
|
||||
cmp_eq(VALUE *a)
|
||||
{
|
||||
VALUE c = rb_exec_recursive_paired_outer(cmp_eq_recursive, a[0], a[1], a[1]);
|
||||
|
||||
if (NIL_P(c)) return Qfalse;
|
||||
if (rb_cmpint(c, a[0], a[1]) == 0) return Qtrue;
|
||||
return Qfalse;
|
||||
}
|
||||
|
||||
static VALUE
|
||||
cmp_failed(void)
|
||||
{
|
||||
rb_warn("Comparable#== will no more rescue exceptions of #<=> in the next release.");
|
||||
rb_warn("Return nil in #<=> if the comparison is inappropriate or avoid such comparison.");
|
||||
return Qfalse;
|
||||
}
|
||||
|
||||
/*
|
||||
* call-seq:
|
||||
* obj == other -> true or false
|
||||
|
@ -73,14 +91,12 @@ cmp_eq_recursive(VALUE arg1, VALUE arg2, int recursive)
|
|||
static VALUE
|
||||
cmp_equal(VALUE x, VALUE y)
|
||||
{
|
||||
VALUE c;
|
||||
VALUE a[2];
|
||||
|
||||
if (x == y) return Qtrue;
|
||||
|
||||
c = rb_exec_recursive_paired_outer(cmp_eq_recursive, x, y, y);
|
||||
|
||||
if (NIL_P(c)) return Qfalse;
|
||||
if (rb_cmpint(c, x, y) == 0) return Qtrue;
|
||||
return Qfalse;
|
||||
a[0] = x; a[1] = y;
|
||||
return rb_rescue(cmp_eq, (VALUE)a, cmp_failed, 0);
|
||||
}
|
||||
|
||||
/*
|
||||
|
|
|
@ -1,4 +1,5 @@
|
|||
require 'test/unit'
|
||||
require_relative 'envutil'
|
||||
|
||||
class TestComparable < Test::Unit::TestCase
|
||||
def setup
|
||||
|
@ -20,8 +21,9 @@ class TestComparable < Test::Unit::TestCase
|
|||
cmp->(x) do raise NotImplementedError, "Not a RuntimeError" end
|
||||
assert_raise(NotImplementedError) { @o == nil }
|
||||
bug7688 = '[ruby-core:51389] [Bug #7688]'
|
||||
cmp->(x) do raise StandardError, "Even a standard error should not be rescued"; end
|
||||
assert_raise(StandardError, bug7688) { @o == nil }
|
||||
cmp->(x) do raise StandardError, "A standard error should be rescued"; end
|
||||
warn = /Comparable#== will no more rescue exceptions .+ in the next release/
|
||||
assert_warn(warn, bug7688) { @o == nil }
|
||||
end
|
||||
|
||||
def test_gt
|
||||
|
|
Loading…
Reference in a new issue