Mesa (master): glcpp: Factor out a tiny bit of repeated code.

Carl Worth cworth at kemper.freedesktop.org
Thu Nov 29 21:08:11 UTC 2012


Module: Mesa
Branch: master
Commit: ea34ac499d46407d8d90baa4a8fd9082ba6b3ea7
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ea34ac499d46407d8d90baa4a8fd9082ba6b3ea7

Author: Carl Worth <cworth at cworth.org>
Date:   Wed Nov 28 12:03:50 2012 -0800

glcpp: Factor out a tiny bit of repeated code.

This function is getting a little long too read. Simplify it by pulling
up one assignment from every condition.

Reviewed-by: Matt Turner <mattst88 at gmail.com>

---

 src/glsl/glcpp/glcpp-parse.y |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y
index d928044..5b322fa 100644
--- a/src/glsl/glcpp/glcpp-parse.y
+++ b/src/glsl/glcpp/glcpp-parse.y
@@ -1528,6 +1528,7 @@ _glcpp_parser_expand_node (glcpp_parser_t *parser,
 		return NULL;
 	}
 
+	*last = node;
 	identifier = token->value.str;
 
 	/* Special handling for __LINE__ and __FILE__, (not through
@@ -1541,7 +1542,6 @@ _glcpp_parser_expand_node (glcpp_parser_t *parser,
 					    node->token->location.first_line);
 		_token_list_append (replacement, value);
 
-		*last = node;
 		return replacement;
 	}
 
@@ -1554,7 +1554,6 @@ _glcpp_parser_expand_node (glcpp_parser_t *parser,
 					    node->token->location.source);
 		_token_list_append (replacement, value);
 
-		*last = node;
 		return replacement;
 	}
 
@@ -1579,14 +1578,12 @@ _glcpp_parser_expand_node (glcpp_parser_t *parser,
 		final = _token_create_str (parser, OTHER, str);
 		expansion = _token_list_create (parser);
 		_token_list_append (expansion, final);
-		*last = node;
 		return expansion;
 	}
 
 	if (! macro->is_function)
 	{
 		token_list_t *replacement;
-		*last = node;
 
 		/* Replace a macro defined as empty with a SPACE token. */
 		if (macro->replacements == NULL)




More information about the mesa-commit mailing list