ruby-changes:49732
From: mame <ko1@a...>
Date: Mon, 15 Jan 2018 17:54:31 +0900 (JST)
Subject: [ruby-changes:49732] mame:r61849 (trunk): parse.y: Removes unneeded NULL checks
mame 2018-01-15 17:54:24 +0900 (Mon, 15 Jan 2018) New Revision: 61849 https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=61849 Log: parse.y: Removes unneeded NULL checks Nowadays, there are less rules whose return value is NULL. Modified files: trunk/parse.y Index: parse.y =================================================================== --- parse.y (revision 61848) +++ parse.y (revision 61849) @@ -1068,13 +1068,7 @@ bodystmt : compstmt https://github.com/ruby/ruby/blob/trunk/parse.y#L1068 $$ = block_append(p, $$, $3); } if ($4) { - if ($$) { - $$ = NEW_ENSURE($$, $4, &@$); - } - else { - NODE *nil = NEW_NIL(&@$); - $$ = block_append(p, $4, nil); - } + $$ = NEW_ENSURE($$, $4, &@$); } fixpos($$, $1); /*% @@ -1375,7 +1369,6 @@ expr_value : expr https://github.com/ruby/ruby/blob/trunk/parse.y#L1369 /*%%%*/ value_expr($1); $$ = $1; - if (!$$) $$ = NEW_NIL(&@$); /*% $$ = $1; %*/ @@ -2107,7 +2100,6 @@ arg_value : arg https://github.com/ruby/ruby/blob/trunk/parse.y#L2100 /*%%%*/ value_expr($1); $$ = $1; - if (!$$) $$ = NEW_NIL(&@$); /*% $$ = $1; %*/ @@ -2800,7 +2792,6 @@ primary_value : primary https://github.com/ruby/ruby/blob/trunk/parse.y#L2792 /*%%%*/ value_expr($1); $$ = $1; - if (!$$) $$ = NEW_NIL(&@$); /*% $$ = $1; %*/ @@ -4524,7 +4515,6 @@ singleton : var_ref https://github.com/ruby/ruby/blob/trunk/parse.y#L4515 /*%%%*/ value_expr($1); $$ = $1; - if (!$$) $$ = NEW_NIL(&@$); /*% $$ = $1; %*/ -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/