[前][次][番号順一覧][スレッド一覧]

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/

[前][次][番号順一覧][スレッド一覧]