diff --git a/ChangeLog b/ChangeLog index 04ab432a0f..ef2114d402 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Tue Jun 17 12:35:24 2014 Nobuyoshi Nakada + + * eval.c (extract_raise_opts): pass unknown options to the + exception, so that exception class can receive a hash argument. + [ruby-core:63203] [Feature #8257] + Tue Jun 17 12:24:57 2014 Koichi Sasada * gc.c (obj_memsize_of): memsize_of(T_ZOMBIE) returns 0, not a rb_bug. diff --git a/eval.c b/eval.c index 5a6c9970d1..ba15de546f 100644 --- a/eval.c +++ b/eval.c @@ -606,12 +606,11 @@ extract_raise_opts(int argc, VALUE *argv, VALUE *opts) if (argc > 0) { VALUE opt = argv[argc-1]; if (RB_TYPE_P(opt, T_HASH)) { - VALUE kw = rb_extract_keywords(&opt); - if (!opt) --argc; - if (kw) { + if (!RHASH_EMPTY_P(opt)) { ID keywords[1]; CONST_ID(keywords[0], "cause"); - rb_get_kwargs(kw, keywords, 0, 1, opts); + rb_get_kwargs(opt, keywords, 0, -1-raise_max_opt, opts); + if (RHASH_EMPTY_P(opt)) --argc; return argc; } } diff --git a/test/ruby/test_exception.rb b/test/ruby/test_exception.rb index 33d10b31d1..b80cf00cac 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -596,4 +596,25 @@ end.join raise cause: cause end end + + def test_unknown_option + bug = '[ruby-core:63203] [Feature #8257] should pass unknown options' + + exc = Class.new(RuntimeError) do + attr_reader :arg + def initialize(msg = nil) + @arg = msg + super(msg) + end + end + + e = assert_raise(exc, bug) {raise exc, "foo" => "bar", foo: "bar"} + assert_equal({"foo" => "bar", foo: "bar"}, e.arg, bug) + + e = assert_raise(exc, bug) {raise exc, "foo" => "bar", foo: "bar", cause: "zzz"} + assert_equal({"foo" => "bar", foo: "bar"}, e.arg, bug) + + e = assert_raise(exc, bug) {raise exc, {}} + assert_equal({}, e.arg, bug) + end end