diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-10-19 17:23:31 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2014-10-19 17:23:31 +0000 |
commit | 6e21b611aaa485259545c0ce696b000aaadee5ed (patch) | |
tree | 1c98379aa5709ce7f20ac84357ffba9438434e50 | |
parent | 11d36a49c7209969043a37e1cf9a666bc9649d8d (diff) | |
download | ruby-6e21b611aaa485259545c0ce696b000aaadee5ed.tar.gz |
variable.c, vm_insnhelper.c: improve performance
* variable.c (rb_ivar_get), vm_insnhelper.c (vm_getivar): improve
instance variable retrieval performance by checking ruby_verbose
before call of rb_warning and evaluation of its argument.
[ruby-core:65786] [Feature #10396]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@48035 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | variable.c | 3 | ||||
-rw-r--r-- | vm_insnhelper.c | 3 |
3 files changed, 11 insertions, 2 deletions
@@ -1,3 +1,10 @@ +Mon Oct 20 02:23:27 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * variable.c (rb_ivar_get), vm_insnhelper.c (vm_getivar): improve + instance variable retrieval performance by checking ruby_verbose + before call of rb_warning and evaluation of its argument. + [ruby-core:65786] [Feature #10396] + Sun Oct 19 23:31:29 2014 CHIKANAGA Tomoyuki <nagachika@ruby-lang.org> * lib/unicode_normalize.rb: (unicode_normalize!): change method name. diff --git a/variable.c b/variable.c index 3364da5a87..006de230fb 100644 --- a/variable.c +++ b/variable.c @@ -1132,7 +1132,8 @@ rb_ivar_get(VALUE obj, ID id) VALUE iv = rb_ivar_lookup(obj, id, Qundef); if (iv == Qundef) { - rb_warning("instance variable %"PRIsVALUE" not initialized", QUOTE_ID(id)); + if (RTEST(ruby_verbose)) + rb_warning("instance variable %"PRIsVALUE" not initialized", QUOTE_ID(id)); iv = Qnil; } return iv; diff --git a/vm_insnhelper.c b/vm_insnhelper.c index 41c1d0678c..b81001b0ef 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -526,7 +526,8 @@ vm_getivar(VALUE obj, ID id, IC ic, rb_call_info_t *ci, int is_attr) } if (UNLIKELY(val == Qundef)) { - if (!is_attr) rb_warning("instance variable %s not initialized", rb_id2name(id)); + if (!is_attr && RTEST(ruby_verbose)) + rb_warning("instance variable %s not initialized", rb_id2name(id)); val = Qnil; } return val; |