Check active expansions before expanding a function-like macro invocation.

With this fix, test 32 no longer recurses infinitely, but now passes.

1 files changed, 5 insertions, 5 deletions

diff --git a/glcpp-parse.y b/glcpp-parse.y
index 60eaf215b8..a2bff6e0ad 100644
--- a/glcpp-parse.y
+++ b/glcpp-parse.y
@@ -654,11 +654,6 @@ _glcpp_parser_print_expanded_token (glcpp_parser_t *parser,
 		return 0;
 	}
 
-	/* For function-like macros return 1 for further processing. */
-	if (macro->is_function) {
-		return 1;
-	}
-
 	/* Finally, don't expand this macro if we're already actively
 	 * expanding it, (to avoid infinite recursion). */
 	if (_string_list_contains (parser->active, identifier, NULL)) {
@@ -666,6 +661,11 @@ _glcpp_parser_print_expanded_token (glcpp_parser_t *parser,
 		return 0;
 	}
 
+	/* For function-like macros return 1 for further processing. */
+	if (macro->is_function) {
+		return 1;
+	}
+
 	_string_list_push (parser->active, identifier);
 	_glcpp_parser_print_expanded_token_list (parser,
 						 macro->replacements);