ruby-changes:21370
From: nobu <ko1@a...>
Date: Thu, 6 Oct 2011 16:29:48 +0900 (JST)
Subject: [ruby-changes:21370] nobu:r33419 (trunk): * vm_eval.c (make_no_method_execption): extract from
nobu 2011-10-06 16:29:33 +0900 (Thu, 06 Oct 2011) New Revision: 33419 http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=33419 Log: * vm_eval.c (make_no_method_execption): extract from raise_method_missing(). * vm_eval.c (send_internal): remove inadvertent symbol creation from public_send. based on a patch by Jeremy Evans <code AT jeremyevans.net> in [ruby-core:38576]. [Feature #5112] * vm_insnhelper.c (vm_call_method): remove inadvertent symbol creation from send and __send__, too. Added directories: trunk/ext/-test-/symbol/ trunk/test/-ext-/symbol/ Added files: trunk/ext/-test-/symbol/extconf.rb trunk/ext/-test-/symbol/init.c trunk/ext/-test-/symbol/intern.c trunk/test/-ext-/symbol/test_inadvertent_creation.rb Modified files: trunk/ChangeLog trunk/vm_eval.c trunk/vm_insnhelper.c trunk/vm_insnhelper.h Index: ChangeLog =================================================================== --- ChangeLog (revision 33418) +++ ChangeLog (revision 33419) @@ -1,3 +1,15 @@ +Thu Oct 6 16:29:30 2011 Nobuyoshi Nakada <nobu@r...> + + * vm_eval.c (make_no_method_execption): extract from + raise_method_missing(). + + * vm_eval.c (send_internal): remove inadvertent symbol creation + from public_send. based on a patch by Jeremy Evans <code AT + jeremyevans.net> in [ruby-core:38576]. [Feature #5112] + + * vm_insnhelper.c (vm_call_method): remove inadvertent symbol + creation from send and __send__, too. + Thu Oct 6 14:59:11 2011 Eric Hodel <drbrain@s...> * lib/time.rb: Clean up Time documentation. Patch by Jake Goulding. Index: vm_eval.c =================================================================== --- vm_eval.c (revision 33418) +++ vm_eval.c (revision 33419) @@ -495,6 +495,30 @@ #define NOEX_MISSING 0x80 +static VALUE +make_no_method_execption(VALUE exc, const char *format, VALUE obj, int argc, const VALUE *argv) +{ + int n = 0; + VALUE mesg; + VALUE args[3]; + + if (!format) { + format = "undefined method `%s' for %s"; + } + mesg = rb_const_get(exc, rb_intern("message")); + if (rb_method_basic_definition_p(CLASS_OF(mesg), '!')) { + args[n++] = rb_name_err_mesg_new(mesg, rb_str_new2(format), obj, argv[0]); + } + else { + args[n++] = rb_funcall(mesg, '!', 3, rb_str_new2(format), obj, argv[0]); + } + args[n++] = argv[0]; + if (exc == rb_eNoMethodError) { + args[n++] = rb_ary_new4(argc - 1, argv + 1); + } + return rb_class_new_instance(n, args, exc); +} + static void raise_method_missing(rb_thread_t *th, int argc, const VALUE *argv, VALUE obj, int last_call_status) @@ -524,28 +548,9 @@ else if (last_call_status & NOEX_SUPER) { format = "super: no superclass method `%s' for %s"; } - if (!format) { - format = "undefined method `%s' for %s"; - } { - int n = 0; - VALUE mesg; - VALUE args[3]; - - mesg = rb_const_get(exc, rb_intern("message")); - if (rb_method_basic_definition_p(CLASS_OF(mesg), '!')) { - args[n++] = rb_name_err_mesg_new(mesg, rb_str_new2(format), obj, argv[0]); - } - else { - args[n++] = rb_funcall(mesg, '!', 3, rb_str_new2(format), obj, argv[0]); - } - args[n++] = argv[0]; - if (exc == rb_eNoMethodError) { - args[n++] = rb_ary_new4(argc - 1, argv + 1); - } - exc = rb_class_new_instance(n, args, exc); - + exc = make_no_method_execption(exc, format, obj, argc, argv); if (!(last_call_status & NOEX_MISSING)) { th->cfp = RUBY_VM_PREVIOUS_CONTROL_FRAME(th->cfp); } @@ -697,6 +702,7 @@ static VALUE send_internal(int argc, const VALUE *argv, VALUE recv, call_type scope) { + ID id; VALUE vid; VALUE self = RUBY_VM_PREVIOUS_CONTROL_FRAME(GET_THREAD()->cfp)->self; rb_thread_t *th = GET_THREAD(); @@ -708,7 +714,16 @@ vid = *argv++; argc--; PASS_PASSED_BLOCK_TH(th); - return rb_call0(recv, rb_to_id(vid), argc, argv, scope, self); + id = rb_check_id(&vid); + if (!id) { + if (rb_method_basic_definition_p(CLASS_OF(recv), idMethodMissing)) { + VALUE exc = make_no_method_execption(rb_eNoMethodError, NULL, + recv, ++argc, --argv); + rb_exc_raise(exc); + } + id = rb_to_id(vid); + } + return rb_call0(recv, id, argc, argv, scope, self); } /* Index: ext/-test-/symbol/intern.c =================================================================== --- ext/-test-/symbol/intern.c (revision 0) +++ ext/-test-/symbol/intern.c (revision 33419) @@ -0,0 +1,14 @@ +#include "ruby.h" + +static VALUE +bug_sym_interned_p(VALUE self, VALUE name) +{ + ID id = rb_check_id(&name); + return id ? Qtrue : Qfalse; +} + +void +Init_intern(VALUE klass) +{ + rb_define_singleton_method(klass, "interned?", bug_sym_interned_p, 1); +} Index: ext/-test-/symbol/init.c =================================================================== --- ext/-test-/symbol/init.c (revision 0) +++ ext/-test-/symbol/init.c (revision 33419) @@ -0,0 +1,11 @@ +#include "ruby.h" + +#define init(n) {void Init_##n(VALUE klass); Init_##n(klass);} + +void +Init_symbol(void) +{ + VALUE mBug = rb_define_module("Bug"); + VALUE klass = rb_define_class_under(mBug, "Symbol", rb_cSymbol); + TEST_INIT_FUNCS(init); +} Index: ext/-test-/symbol/extconf.rb =================================================================== --- ext/-test-/symbol/extconf.rb (revision 0) +++ ext/-test-/symbol/extconf.rb (revision 33419) @@ -0,0 +1,6 @@ +$srcs = Dir[File.join($srcdir, "*.{#{SRC_EXT.join(%q{,})}}")] +inits = $srcs.map {|s| File.basename(s, ".*")} +inits.delete("init") +inits.map! {|s|"X(#{s})"} +$defs << "-DTEST_INIT_FUNCS(X)=\"#{inits.join(' ')}\"" +create_makefile("-test-/symbol/symbol") Index: vm_insnhelper.c =================================================================== --- vm_insnhelper.c (revision 33418) +++ vm_insnhelper.c (revision 33419) @@ -584,7 +584,17 @@ } sym = TOPN(i); - id = SYMBOL_P(sym) ? SYM2ID(sym) : rb_to_id(sym); + if (SYMBOL_P(sym)) { + id = SYM2ID(sym); + } + else if (!(id = rb_check_id(&sym))) { + if (rb_method_basic_definition_p(CLASS_OF(recv), idMethodMissing)) { + VALUE exc = make_no_method_execption(rb_eNoMethodError, NULL, recv, + rb_long2int(num), &TOPN(i)); + rb_exc_raise(exc); + } + id = rb_to_id(sym); + } /* shift arguments */ if (i > 0) { MEMMOVE(&TOPN(i), &TOPN(i-1), VALUE, i); Index: vm_insnhelper.h =================================================================== --- vm_insnhelper.h (revision 33418) +++ vm_insnhelper.h (revision 33419) @@ -226,4 +226,8 @@ } while (0) static void vm_clear_all_cache(void); +static VALUE make_no_method_execption(VALUE exc, const char *format, + VALUE obj, int argc, const VALUE *argv); + + #endif /* RUBY_INSNHELPER_H */ Index: test/-ext-/symbol/test_inadvertent_creation.rb =================================================================== --- test/-ext-/symbol/test_inadvertent_creation.rb (revision 0) +++ test/-ext-/symbol/test_inadvertent_creation.rb (revision 33419) @@ -0,0 +1,44 @@ +require 'test/unit' +require "-test-/symbol/symbol" + +module Test_Symbol + class TestInadvertent < Test::Unit::TestCase + def self.noninterned_name + th = Thread.current.object_id.to_s(36) + begin + name = "#{th}.#{rand(0x1000).to_s(16)}.#{Time.now.usec}" + end while Bug::Symbol.interned?(name) + name + end + + def setup + @obj = Object.new + end + + Feature5112 = '[ruby-core:38576]' + + def test_public_send + name = self.class.noninterned_name + e = assert_raise(NoMethodError) {@obj.public_send(name, Feature5112)} + assert_not_send([Bug::Symbol, :interned?, name]) + assert_equal(name, e.name) + assert_equal([Feature5112], e.args) + end + + def test_send + name = self.class.noninterned_name + e = assert_raise(NoMethodError) {@obj.send(name, Feature5112)} + assert_not_send([Bug::Symbol, :interned?, name]) + assert_equal(name, e.name) + assert_equal([Feature5112], e.args) + end + + def test___send__ + name = self.class.noninterned_name + e = assert_raise(NoMethodError) {@obj.__send__(name, Feature5112)} + assert_not_send([Bug::Symbol, :interned?, name]) + assert_equal(name, e.name) + assert_equal([Feature5112], e.args) + end + end +end -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/