diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | error.c | 3 |
2 files changed, 8 insertions, 1 deletions
@@ -1,3 +1,9 @@ +Thu Jun 20 13:14:06 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * error.c (rb_check_backtrace): evaluate RARRAY_AREF only once. + the first argument of RB_TYPE_P is expanded twice for non-immediate + types. + Thu Jun 20 08:09:29 2013 Koichi Sasada <ko1@atdot.net> * tool/gcbench.rb: Summary in one line. @@ -721,7 +721,8 @@ rb_check_backtrace(VALUE bt) rb_raise(rb_eTypeError, err); } for (i=0;i<RARRAY_LEN(bt);i++) { - if (!RB_TYPE_P(RARRAY_AREF(bt, i), T_STRING)) { + VALUE e = RARRAY_AREF(bt, i); + if (!RB_TYPE_P(e, T_STRING)) { rb_raise(rb_eTypeError, err); } } |