From e0611820df64b7cc70cbe15535b82e245913ee97 Mon Sep 17 00:00:00 2001 From: nobu Date: Tue, 3 May 2016 05:19:20 +0000 Subject: [PATCH] parse.y: trace elsif * parse.y (new_if_gen): set newline flag to NODE_IF to trace all if/elsif statements. [ruby-core:67720] [Bug #10763] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@54893 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 5 +++++ parse.y | 2 +- test/ruby/test_settracefunc.rb | 19 +++++++++++++++++++ 3 files changed, 25 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 7a5307fef1..267b3433c0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,8 @@ +Tue May 3 14:19:18 2016 Nobuyoshi Nakada + + * parse.y (new_if_gen): set newline flag to NODE_IF to trace all + if/elsif statements. [ruby-core:67720] [Bug #10763] + Tue May 3 05:35:54 2016 Eric Wong * process.c (disable_child_handler_fork_child): diff --git a/parse.y b/parse.y index e3facb73c8..005e926ef0 100644 --- a/parse.y +++ b/parse.y @@ -9955,7 +9955,7 @@ new_if_gen(struct parser_params *parser, NODE *cc, NODE *left, NODE *right) { if (!cc) return right; cc = cond0(parser, cc); - return NEW_IF(cc, left, right); + return newline_node(NEW_IF(cc, left, right)); } static NODE* diff --git a/test/ruby/test_settracefunc.rb b/test/ruby/test_settracefunc.rb index 9de2137aab..018c0062c3 100644 --- a/test/ruby/test_settracefunc.rb +++ b/test/ruby/test_settracefunc.rb @@ -1399,6 +1399,25 @@ class TestSetTraceFunc < Test::Unit::TestCase assert_equal [__LINE__ - 3, __LINE__ - 2], lines, 'Bug #10449' end + def test_elsif_line_event + bug10763 = '[ruby-core:67720] [Bug #10763]' + lines = [] + line = nil + + TracePoint.new(:line){|tp| + next unless target_thread? + lines << tp.lineno if line + }.enable{ + line = __LINE__ + if !line + 1 + elsif line + 2 + end + } + assert_equal [line+1, line+3, line+4], lines, bug10763 + end + class Bug10724 def initialize loop{return}