[Mesa-dev] [PATCH] glsl: remove tabs and fix some other style issues in glcpp-parse.y

Timothy Arceri timothy.arceri at collabora.com
Thu Mar 31 05:53:30 UTC 2016


Note there are still tabs left in the parser rules.
---
 src/compiler/glsl/glcpp/glcpp-parse.y | 2754 ++++++++++++++++-----------------
 1 file changed, 1341 insertions(+), 1413 deletions(-)

diff --git a/src/compiler/glsl/glcpp/glcpp-parse.y b/src/compiler/glsl/glcpp/glcpp-parse.y
index fbbf85b..8a4f6da 100644
--- a/src/compiler/glsl/glcpp/glcpp-parse.y
+++ b/src/compiler/glsl/glcpp/glcpp-parse.y
@@ -33,125 +33,121 @@
 #include "main/mtypes.h" /* for gl_api enum */
 
 static void
-yyerror (YYLTYPE *locp, glcpp_parser_t *parser, const char *error);
+yyerror(YYLTYPE *locp, glcpp_parser_t *parser, const char *error);
 
 static void
-_define_object_macro (glcpp_parser_t *parser,
-		      YYLTYPE *loc,
-		      const char *macro,
-		      token_list_t *replacements);
+_define_object_macro(glcpp_parser_t *parser,
+                     YYLTYPE *loc,
+                     const char *macro,
+                     token_list_t *replacements);
 
 static void
-_define_function_macro (glcpp_parser_t *parser,
-			YYLTYPE *loc,
-			const char *macro,
-			string_list_t *parameters,
-			token_list_t *replacements);
+_define_function_macro(glcpp_parser_t *parser,
+                       YYLTYPE *loc,
+                       const char *macro,
+                       string_list_t *parameters,
+                       token_list_t *replacements);
 
 static string_list_t *
-_string_list_create (void *ctx);
+_string_list_create(void *ctx);
 
 static void
-_string_list_append_item (string_list_t *list, const char *str);
+_string_list_append_item(string_list_t *list, const char *str);
 
 static int
-_string_list_contains (string_list_t *list, const char *member, int *index);
+_string_list_contains(string_list_t *list, const char *member, int *index);
 
 static const char *
-_string_list_has_duplicate (string_list_t *list);
+_string_list_has_duplicate(string_list_t *list);
 
 static int
-_string_list_length (string_list_t *list);
+_string_list_length(string_list_t *list);
 
 static int
-_string_list_equal (string_list_t *a, string_list_t *b);
+_string_list_equal(string_list_t *a, string_list_t *b);
 
 static argument_list_t *
-_argument_list_create (void *ctx);
+_argument_list_create(void *ctx);
 
 static void
-_argument_list_append (argument_list_t *list, token_list_t *argument);
+_argument_list_append(argument_list_t *list, token_list_t *argument);
 
 static int
-_argument_list_length (argument_list_t *list);
+_argument_list_length(argument_list_t *list);
 
 static token_list_t *
-_argument_list_member_at (argument_list_t *list, int index);
+_argument_list_member_at(argument_list_t *list, int index);
 
 /* Note: This function ralloc_steal()s the str pointer. */
 static token_t *
-_token_create_str (void *ctx, int type, char *str);
+_token_create_str(void *ctx, int type, char *str);
 
 static token_t *
-_token_create_ival (void *ctx, int type, int ival);
+_token_create_ival(void *ctx, int type, int ival);
 
 static token_list_t *
-_token_list_create (void *ctx);
+_token_list_create(void *ctx);
 
 static void
-_token_list_append (token_list_t *list, token_t *token);
+_token_list_append(token_list_t *list, token_t *token);
 
 static void
-_token_list_append_list (token_list_t *list, token_list_t *tail);
+_token_list_append_list(token_list_t *list, token_list_t *tail);
 
 static int
-_token_list_equal_ignoring_space (token_list_t *a, token_list_t *b);
+_token_list_equal_ignoring_space(token_list_t *a, token_list_t *b);
 
 static void
-_parser_active_list_push (glcpp_parser_t *parser,
-			  const char *identifier,
-			  token_node_t *marker);
+_parser_active_list_push(glcpp_parser_t *parser, const char *identifier,
+                         token_node_t *marker);
 
 static void
-_parser_active_list_pop (glcpp_parser_t *parser);
+_parser_active_list_pop(glcpp_parser_t *parser);
 
 static int
-_parser_active_list_contains (glcpp_parser_t *parser, const char *identifier);
+_parser_active_list_contains(glcpp_parser_t *parser, const char *identifier);
 
 typedef enum {
-	EXPANSION_MODE_IGNORE_DEFINED,
-	EXPANSION_MODE_EVALUATE_DEFINED
+   EXPANSION_MODE_IGNORE_DEFINED,
+   EXPANSION_MODE_EVALUATE_DEFINED
 } expansion_mode_t;
 
 /* Expand list, and begin lexing from the result (after first
  * prefixing a token of type 'head_token_type').
  */
 static void
-_glcpp_parser_expand_and_lex_from (glcpp_parser_t *parser,
-				   int head_token_type,
-				   token_list_t *list,
-				   expansion_mode_t mode);
+_glcpp_parser_expand_and_lex_from(glcpp_parser_t *parser, int head_token_type,
+                                  token_list_t *list, expansion_mode_t mode);
 
 /* Perform macro expansion in-place on the given list. */
 static void
-_glcpp_parser_expand_token_list (glcpp_parser_t *parser,
-				 token_list_t *list,
-				 expansion_mode_t mode);
+_glcpp_parser_expand_token_list(glcpp_parser_t *parser, token_list_t *list,
+                                expansion_mode_t mode);
 
 static void
-_glcpp_parser_print_expanded_token_list (glcpp_parser_t *parser,
-					 token_list_t *list);
+_glcpp_parser_print_expanded_token_list(glcpp_parser_t *parser,
+                                        token_list_t *list);
 
 static void
-_glcpp_parser_skip_stack_push_if (glcpp_parser_t *parser, YYLTYPE *loc,
-				  int condition);
+_glcpp_parser_skip_stack_push_if(glcpp_parser_t *parser, YYLTYPE *loc,
+                                 int condition);
 
 static void
-_glcpp_parser_skip_stack_change_if (glcpp_parser_t *parser, YYLTYPE *loc,
-				    const char *type, int condition);
+_glcpp_parser_skip_stack_change_if(glcpp_parser_t *parser, YYLTYPE *loc,
+                                   const char *type, int condition);
 
 static void
-_glcpp_parser_skip_stack_pop (glcpp_parser_t *parser, YYLTYPE *loc);
+_glcpp_parser_skip_stack_pop(glcpp_parser_t *parser, YYLTYPE *loc);
 
 static void
 _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t version,
                                          const char *ident, bool explicitly_set);
 
 static int
-glcpp_parser_lex (YYSTYPE *yylval, YYLTYPE *yylloc, glcpp_parser_t *parser);
+glcpp_parser_lex(YYSTYPE *yylval, YYLTYPE *yylloc, glcpp_parser_t *parser);
 
 static void
-glcpp_parser_lex_from (glcpp_parser_t *parser, token_list_t *list);
+glcpp_parser_lex_from(glcpp_parser_t *parser, token_list_t *list);
 
 static void
 add_builtin_define(glcpp_parser_t *parser, const char *name, int value);
@@ -163,11 +159,11 @@ add_builtin_define(glcpp_parser_t *parser, const char *name, int value);
 
 %locations
 %initial-action {
-	@$.first_line = 1;
-	@$.first_column = 1;
-	@$.last_line = 1;
-	@$.last_column = 1;
-	@$.source = 0;
+   @$.first_line = 1;
+   @$.first_column = 1;
+   @$.last_line = 1;
+   @$.last_column = 1;
+   @$.source = 0;
 }
 
 %parse-param {glcpp_parser_t *parser}
@@ -175,7 +171,7 @@ add_builtin_define(glcpp_parser_t *parser, const char *name, int value);
 
 %expect 0
 
-	/* We use HASH_TOKEN, DEFINE_TOKEN and VERSION_TOKEN (as opposed to
+        /* We use HASH_TOKEN, DEFINE_TOKEN and VERSION_TOKEN (as opposed to
          * HASH, DEFINE, and VERSION) to avoid conflicts with other symbols,
          * (such as the <HASH> and <DEFINE> start conditions in the lexer). */
 %token DEFINED ELIF_EXPANDED HASH_TOKEN DEFINE_TOKEN FUNC_IDENTIFIER OBJ_IDENTIFIER ELIF ELSE ENDIF ERROR_TOKEN IF IFDEF IFNDEF LINE PRAGMA UNDEF VERSION_TOKEN GARBAGE IDENTIFIER IF_EXPANDED INTEGER INTEGER_STRING LINE_EXPANDED NEWLINE OTHER PLACEHOLDER SPACE PLUS_PLUS MINUS_MINUS
@@ -730,438 +726,433 @@ operator:
 %%
 
 string_list_t *
-_string_list_create (void *ctx)
+_string_list_create(void *ctx)
 {
-	string_list_t *list;
+   string_list_t *list;
 
-	list = ralloc (ctx, string_list_t);
-	list->head = NULL;
-	list->tail = NULL;
+   list = ralloc (ctx, string_list_t);
+   list->head = NULL;
+   list->tail = NULL;
 
-	return list;
+   return list;
 }
 
 void
-_string_list_append_item (string_list_t *list, const char *str)
+_string_list_append_item(string_list_t *list, const char *str)
 {
-	string_node_t *node;
+   string_node_t *node;
 
-	node = ralloc (list, string_node_t);
-	node->str = ralloc_strdup (node, str);
+   node = ralloc (list, string_node_t);
+   node->str = ralloc_strdup (node, str);
 
-	node->next = NULL;
+   node->next = NULL;
 
-	if (list->head == NULL) {
-		list->head = node;
-	} else {
-		list->tail->next = node;
-	}
+   if (list->head == NULL) {
+      list->head = node;
+   } else {
+      list->tail->next = node;
+   }
 
-	list->tail = node;
+   list->tail = node;
 }
 
 int
-_string_list_contains (string_list_t *list, const char *member, int *index)
+_string_list_contains(string_list_t *list, const char *member, int *index)
 {
-	string_node_t *node;
-	int i;
+   string_node_t *node;
+   int i;
 
-	if (list == NULL)
-		return 0;
+   if (list == NULL)
+      return 0;
 
-	for (i = 0, node = list->head; node; i++, node = node->next) {
-		if (strcmp (node->str, member) == 0) {
-			if (index)
-				*index = i;
-			return 1;
-		}
-	}
+   for (i = 0, node = list->head; node; i++, node = node->next) {
+      if (strcmp (node->str, member) == 0) {
+         if (index)
+            *index = i;
+         return 1;
+      }
+   }
 
-	return 0;
+   return 0;
 }
 
 /* Return duplicate string in list (if any), NULL otherwise. */
 const char *
-_string_list_has_duplicate (string_list_t *list)
+_string_list_has_duplicate(string_list_t *list)
 {
-	string_node_t *node, *dup;
+   string_node_t *node, *dup;
 
-	if (list == NULL)
-		return NULL;
+   if (list == NULL)
+      return NULL;
 
-	for (node = list->head; node; node = node->next) {
-		for (dup = node->next; dup; dup = dup->next) {
-			if (strcmp (node->str, dup->str) == 0)
-				return node->str;
-		}
-	}
+   for (node = list->head; node; node = node->next) {
+      for (dup = node->next; dup; dup = dup->next) {
+         if (strcmp (node->str, dup->str) == 0)
+            return node->str;
+      }
+   }
 
-	return NULL;
+   return NULL;
 }
 
 int
-_string_list_length (string_list_t *list)
+_string_list_length(string_list_t *list)
 {
-	int length = 0;
-	string_node_t *node;
+   int length = 0;
+   string_node_t *node;
 
-	if (list == NULL)
-		return 0;
+   if (list == NULL)
+      return 0;
 
-	for (node = list->head; node; node = node->next)
-		length++;
+   for (node = list->head; node; node = node->next)
+      length++;
 
-	return length;
+   return length;
 }
 
 int
-_string_list_equal (string_list_t *a, string_list_t *b)
+_string_list_equal(string_list_t *a, string_list_t *b)
 {
-	string_node_t *node_a, *node_b;
-
-	if (a == NULL && b == NULL)
-		return 1;
-
-	if (a == NULL || b == NULL)
-		return 0;
-
-	for (node_a = a->head, node_b = b->head;
-	     node_a && node_b;
-	     node_a = node_a->next, node_b = node_b->next)
-	{
-		if (strcmp (node_a->str, node_b->str))
-			return 0;
-	}
-
-	/* Catch the case of lists being different lengths, (which
-	 * would cause the loop above to terminate after the shorter
-	 * list). */
-	return node_a == node_b;
+   string_node_t *node_a, *node_b;
+
+   if (a == NULL && b == NULL)
+      return 1;
+
+   if (a == NULL || b == NULL)
+      return 0;
+
+   for (node_a = a->head, node_b = b->head;
+        node_a && node_b;
+        node_a = node_a->next, node_b = node_b->next)
+   {
+      if (strcmp (node_a->str, node_b->str))
+         return 0;
+   }
+
+   /* Catch the case of lists being different lengths, (which
+    * would cause the loop above to terminate after the shorter
+    * list). */
+   return node_a == node_b;
 }
 
 argument_list_t *
-_argument_list_create (void *ctx)
+_argument_list_create(void *ctx)
 {
-	argument_list_t *list;
+   argument_list_t *list;
 
-	list = ralloc (ctx, argument_list_t);
-	list->head = NULL;
-	list->tail = NULL;
+   list = ralloc (ctx, argument_list_t);
+   list->head = NULL;
+   list->tail = NULL;
 
-	return list;
+   return list;
 }
 
 void
-_argument_list_append (argument_list_t *list, token_list_t *argument)
+_argument_list_append(argument_list_t *list, token_list_t *argument)
 {
-	argument_node_t *node;
+   argument_node_t *node;
 
-	node = ralloc (list, argument_node_t);
-	node->argument = argument;
+   node = ralloc (list, argument_node_t);
+   node->argument = argument;
 
-	node->next = NULL;
+   node->next = NULL;
 
-	if (list->head == NULL) {
-		list->head = node;
-	} else {
-		list->tail->next = node;
-	}
+   if (list->head == NULL) {
+      list->head = node;
+   } else {
+      list->tail->next = node;
+   }
 
-	list->tail = node;
+   list->tail = node;
 }
 
 int
-_argument_list_length (argument_list_t *list)
+_argument_list_length(argument_list_t *list)
 {
-	int length = 0;
-	argument_node_t *node;
+   int length = 0;
+   argument_node_t *node;
 
-	if (list == NULL)
-		return 0;
+   if (list == NULL)
+      return 0;
 
-	for (node = list->head; node; node = node->next)
-		length++;
+   for (node = list->head; node; node = node->next)
+      length++;
 
-	return length;
+   return length;
 }
 
 token_list_t *
-_argument_list_member_at (argument_list_t *list, int index)
+_argument_list_member_at(argument_list_t *list, int index)
 {
-	argument_node_t *node;
-	int i;
+   argument_node_t *node;
+   int i;
 
-	if (list == NULL)
-		return NULL;
+   if (list == NULL)
+      return NULL;
 
-	node = list->head;
-	for (i = 0; i < index; i++) {
-		node = node->next;
-		if (node == NULL)
-			break;
-	}
+   node = list->head;
+   for (i = 0; i < index; i++) {
+      node = node->next;
+      if (node == NULL)
+         break;
+   }
 
-	if (node)
-		return node->argument;
+   if (node)
+      return node->argument;
 
-	return NULL;
+   return NULL;
 }
 
 /* Note: This function ralloc_steal()s the str pointer. */
 token_t *
-_token_create_str (void *ctx, int type, char *str)
+_token_create_str(void *ctx, int type, char *str)
 {
-	token_t *token;
+   token_t *token;
 
-	token = ralloc (ctx, token_t);
-	token->type = type;
-	token->value.str = str;
+   token = ralloc (ctx, token_t);
+   token->type = type;
+   token->value.str = str;
 
-	ralloc_steal (token, str);
+   ralloc_steal (token, str);
 
-	return token;
+   return token;
 }
 
 token_t *
-_token_create_ival (void *ctx, int type, int ival)
+_token_create_ival(void *ctx, int type, int ival)
 {
-	token_t *token;
+   token_t *token;
 
-	token = ralloc (ctx, token_t);
-	token->type = type;
-	token->value.ival = ival;
+   token = ralloc (ctx, token_t);
+   token->type = type;
+   token->value.ival = ival;
 
-	return token;
+   return token;
 }
 
 token_list_t *
-_token_list_create (void *ctx)
+_token_list_create(void *ctx)
 {
-	token_list_t *list;
+   token_list_t *list;
 
-	list = ralloc (ctx, token_list_t);
-	list->head = NULL;
-	list->tail = NULL;
-	list->non_space_tail = NULL;
+   list = ralloc (ctx, token_list_t);
+   list->head = NULL;
+   list->tail = NULL;
+   list->non_space_tail = NULL;
 
-	return list;
+   return list;
 }
 
 void
-_token_list_append (token_list_t *list, token_t *token)
+_token_list_append(token_list_t *list, token_t *token)
 {
-	token_node_t *node;
+   token_node_t *node;
 
-	node = ralloc (list, token_node_t);
-	node->token = token;
-	node->next = NULL;
+   node = ralloc (list, token_node_t);
+   node->token = token;
+   node->next = NULL;
 
-	if (list->head == NULL) {
-		list->head = node;
-	} else {
-		list->tail->next = node;
-	}
+   if (list->head == NULL) {
+      list->head = node;
+   } else {
+      list->tail->next = node;
+   }
 
-	list->tail = node;
-	if (token->type != SPACE)
-		list->non_space_tail = node;
+   list->tail = node;
+   if (token->type != SPACE)
+      list->non_space_tail = node;
 }
 
 void
-_token_list_append_list (token_list_t *list, token_list_t *tail)
+_token_list_append_list(token_list_t *list, token_list_t *tail)
 {
-	if (tail == NULL || tail->head == NULL)
-		return;
+   if (tail == NULL || tail->head == NULL)
+      return;
 
-	if (list->head == NULL) {
-		list->head = tail->head;
-	} else {
-		list->tail->next = tail->head;
-	}
+   if (list->head == NULL) {
+      list->head = tail->head;
+   } else {
+      list->tail->next = tail->head;
+   }
 
-	list->tail = tail->tail;
-	list->non_space_tail = tail->non_space_tail;
+   list->tail = tail->tail;
+   list->non_space_tail = tail->non_space_tail;
 }
 
 static token_list_t *
-_token_list_copy (void *ctx, token_list_t *other)
+_token_list_copy(void *ctx, token_list_t *other)
 {
-	token_list_t *copy;
-	token_node_t *node;
+   token_list_t *copy;
+   token_node_t *node;
 
-	if (other == NULL)
-		return NULL;
+   if (other == NULL)
+      return NULL;
 
-	copy = _token_list_create (ctx);
-	for (node = other->head; node; node = node->next) {
-		token_t *new_token = ralloc (copy, token_t);
-		*new_token = *node->token;
-		_token_list_append (copy, new_token);
-	}
+   copy = _token_list_create (ctx);
+   for (node = other->head; node; node = node->next) {
+      token_t *new_token = ralloc (copy, token_t);
+      *new_token = *node->token;
+      _token_list_append (copy, new_token);
+   }
 
-	return copy;
+   return copy;
 }
 
 static void
-_token_list_trim_trailing_space (token_list_t *list)
+_token_list_trim_trailing_space(token_list_t *list)
 {
-	token_node_t *tail, *next;
-
-	if (list->non_space_tail) {
-		tail = list->non_space_tail->next;
-		list->non_space_tail->next = NULL;
-		list->tail = list->non_space_tail;
-
-		while (tail) {
-			next = tail->next;
-			ralloc_free (tail);
-			tail = next;
-		}
-	}
+   token_node_t *tail, *next;
+
+   if (list->non_space_tail) {
+      tail = list->non_space_tail->next;
+      list->non_space_tail->next = NULL;
+      list->tail = list->non_space_tail;
+
+      while (tail) {
+         next = tail->next;
+         ralloc_free (tail);
+         tail = next;
+      }
+   }
 }
 
 static int
-_token_list_is_empty_ignoring_space (token_list_t *l)
+_token_list_is_empty_ignoring_space(token_list_t *l)
 {
-	token_node_t *n;
+   token_node_t *n;
 
-	if (l == NULL)
-		return 1;
+   if (l == NULL)
+      return 1;
 
-	n = l->head;
-	while (n != NULL && n->token->type == SPACE)
-		n = n->next;
+   n = l->head;
+   while (n != NULL && n->token->type == SPACE)
+      n = n->next;
 
-	return n == NULL;
+   return n == NULL;
 }
 
 int
-_token_list_equal_ignoring_space (token_list_t *a, token_list_t *b)
+_token_list_equal_ignoring_space(token_list_t *a, token_list_t *b)
 {
-	token_node_t *node_a, *node_b;
-
-	if (a == NULL || b == NULL) {
-		int a_empty = _token_list_is_empty_ignoring_space(a);
-		int b_empty = _token_list_is_empty_ignoring_space(b);
-		return a_empty == b_empty;
-	}
-
-	node_a = a->head;
-	node_b = b->head;
-
-	while (1)
-	{
-		if (node_a == NULL && node_b == NULL)
-			break;
-
-		if (node_a == NULL || node_b == NULL)
-			return 0;
-		/* Make sure whitespace appears in the same places in both.
-		 * It need not be exactly the same amount of whitespace,
-		 * though.
-		 */
-		if (node_a->token->type == SPACE
-		    && node_b->token->type == SPACE) {
-			while (node_a && node_a->token->type == SPACE)
-				node_a = node_a->next;
-			while (node_b && node_b->token->type == SPACE)
-				node_b = node_b->next;
-			continue;
-		}
-
-		if (node_a->token->type != node_b->token->type)
-			return 0;
-
-		switch (node_a->token->type) {
-		case INTEGER:
-			if (node_a->token->value.ival != 
-			    node_b->token->value.ival)
-			{
-				return 0;
-			}
-			break;
-		case IDENTIFIER:
-		case INTEGER_STRING:
-		case OTHER:
-			if (strcmp (node_a->token->value.str,
-				    node_b->token->value.str))
-			{
-				return 0;
-			}
-			break;
-		}
-
-		node_a = node_a->next;
-		node_b = node_b->next;
-	}
-
-	return 1;
+   token_node_t *node_a, *node_b;
+
+   if (a == NULL || b == NULL) {
+      int a_empty = _token_list_is_empty_ignoring_space(a);
+      int b_empty = _token_list_is_empty_ignoring_space(b);
+      return a_empty == b_empty;
+   }
+
+   node_a = a->head;
+   node_b = b->head;
+
+   while (1)
+   {
+      if (node_a == NULL && node_b == NULL)
+         break;
+
+      if (node_a == NULL || node_b == NULL)
+         return 0;
+      /* Make sure whitespace appears in the same places in both.
+       * It need not be exactly the same amount of whitespace,
+       * though.
+       */
+      if (node_a->token->type == SPACE && node_b->token->type == SPACE) {
+         while (node_a && node_a->token->type == SPACE)
+            node_a = node_a->next;
+         while (node_b && node_b->token->type == SPACE)
+            node_b = node_b->next;
+         continue;
+      }
+
+      if (node_a->token->type != node_b->token->type)
+         return 0;
+
+      switch (node_a->token->type) {
+      case INTEGER:
+         if (node_a->token->value.ival !=  node_b->token->value.ival) {
+            return 0;
+         }
+         break;
+      case IDENTIFIER:
+      case INTEGER_STRING:
+      case OTHER:
+         if (strcmp(node_a->token->value.str, node_b->token->value.str)) {
+            return 0;
+         }
+         break;
+      }
+
+      node_a = node_a->next;
+      node_b = node_b->next;
+   }
+
+   return 1;
 }
 
 static void
-_token_print (char **out, size_t *len, token_t *token)
+_token_print(char **out, size_t *len, token_t *token)
 {
-	if (token->type < 256) {
-		ralloc_asprintf_rewrite_tail (out, len, "%c", token->type);
-		return;
-	}
-
-	switch (token->type) {
-	case INTEGER:
-		ralloc_asprintf_rewrite_tail (out, len, "%" PRIiMAX, token->value.ival);
-		break;
-	case IDENTIFIER:
-	case INTEGER_STRING:
-	case OTHER:
-		ralloc_asprintf_rewrite_tail (out, len, "%s", token->value.str);
-		break;
-	case SPACE:
-		ralloc_asprintf_rewrite_tail (out, len, " ");
-		break;
-	case LEFT_SHIFT:
-		ralloc_asprintf_rewrite_tail (out, len, "<<");
-		break;
-	case RIGHT_SHIFT:
-		ralloc_asprintf_rewrite_tail (out, len, ">>");
-		break;
-	case LESS_OR_EQUAL:
-		ralloc_asprintf_rewrite_tail (out, len, "<=");
-		break;
-	case GREATER_OR_EQUAL:
-		ralloc_asprintf_rewrite_tail (out, len, ">=");
-		break;
-	case EQUAL:
-		ralloc_asprintf_rewrite_tail (out, len, "==");
-		break;
-	case NOT_EQUAL:
-		ralloc_asprintf_rewrite_tail (out, len, "!=");
-		break;
-	case AND:
-		ralloc_asprintf_rewrite_tail (out, len, "&&");
-		break;
-	case OR:
-		ralloc_asprintf_rewrite_tail (out, len, "||");
-		break;
-	case PASTE:
-		ralloc_asprintf_rewrite_tail (out, len, "##");
-		break;
-        case PLUS_PLUS:
-		ralloc_asprintf_rewrite_tail (out, len, "++");
-		break;
-        case MINUS_MINUS:
-		ralloc_asprintf_rewrite_tail (out, len, "--");
-		break;
-	case DEFINED:
-		ralloc_asprintf_rewrite_tail (out, len, "defined");
-		break;
-	case PLACEHOLDER:
-		/* Nothing to print. */
-		break;
-	default:
-		assert(!"Error: Don't know how to print token.");
-
-		break;
-	}
+   if (token->type < 256) {
+      ralloc_asprintf_rewrite_tail (out, len, "%c", token->type);
+      return;
+   }
+
+   switch (token->type) {
+   case INTEGER:
+      ralloc_asprintf_rewrite_tail (out, len, "%" PRIiMAX, token->value.ival);
+      break;
+   case IDENTIFIER:
+   case INTEGER_STRING:
+   case OTHER:
+      ralloc_asprintf_rewrite_tail (out, len, "%s", token->value.str);
+      break;
+   case SPACE:
+      ralloc_asprintf_rewrite_tail (out, len, " ");
+      break;
+   case LEFT_SHIFT:
+      ralloc_asprintf_rewrite_tail (out, len, "<<");
+      break;
+   case RIGHT_SHIFT:
+      ralloc_asprintf_rewrite_tail (out, len, ">>");
+      break;
+   case LESS_OR_EQUAL:
+      ralloc_asprintf_rewrite_tail (out, len, "<=");
+      break;
+   case GREATER_OR_EQUAL:
+      ralloc_asprintf_rewrite_tail (out, len, ">=");
+      break;
+   case EQUAL:
+      ralloc_asprintf_rewrite_tail (out, len, "==");
+      break;
+   case NOT_EQUAL:
+      ralloc_asprintf_rewrite_tail (out, len, "!=");
+      break;
+   case AND:
+      ralloc_asprintf_rewrite_tail (out, len, "&&");
+      break;
+   case OR:
+      ralloc_asprintf_rewrite_tail (out, len, "||");
+      break;
+   case PASTE:
+      ralloc_asprintf_rewrite_tail (out, len, "##");
+      break;
+   case PLUS_PLUS:
+      ralloc_asprintf_rewrite_tail (out, len, "++");
+      break;
+   case MINUS_MINUS:
+      ralloc_asprintf_rewrite_tail (out, len, "--");
+      break;
+   case DEFINED:
+      ralloc_asprintf_rewrite_tail (out, len, "defined");
+      break;
+   case PLACEHOLDER:
+      /* Nothing to print. */
+      break;
+   default:
+      assert(!"Error: Don't know how to print token.");
+
+      break;
+   }
 }
 
 /* Return a new token (ralloc()ed off of 'token') formed by pasting
@@ -1171,148 +1162,143 @@ _token_print (char **out, size_t *len, token_t *token)
  * Caution: Only very cursory error-checking is performed to see if
  * the final result is a valid single token. */
 static token_t *
-_token_paste (glcpp_parser_t *parser, token_t *token, token_t *other)
+_token_paste(glcpp_parser_t *parser, token_t *token, token_t *other)
 {
-	token_t *combined = NULL;
-
-	/* Pasting a placeholder onto anything makes no change. */
-	if (other->type == PLACEHOLDER)
-		return token;
-
-	/* When 'token' is a placeholder, just return 'other'. */
-	if (token->type == PLACEHOLDER)
-		return other;
-
-	/* A very few single-character punctuators can be combined
-	 * with another to form a multi-character punctuator. */
-	switch (token->type) {
-	case '<':
-		if (other->type == '<')
-			combined = _token_create_ival (token, LEFT_SHIFT, LEFT_SHIFT);
-		else if (other->type == '=')
-			combined = _token_create_ival (token, LESS_OR_EQUAL, LESS_OR_EQUAL);
-		break;
-	case '>':
-		if (other->type == '>')
-			combined = _token_create_ival (token, RIGHT_SHIFT, RIGHT_SHIFT);
-		else if (other->type == '=')
-			combined = _token_create_ival (token, GREATER_OR_EQUAL, GREATER_OR_EQUAL);
-		break;
-	case '=':
-		if (other->type == '=')
-			combined = _token_create_ival (token, EQUAL, EQUAL);
-		break;
-	case '!':
-		if (other->type == '=')
-			combined = _token_create_ival (token, NOT_EQUAL, NOT_EQUAL);
-		break;
-	case '&':
-		if (other->type == '&')
-			combined = _token_create_ival (token, AND, AND);
-		break;
-	case '|':
-		if (other->type == '|')
-			combined = _token_create_ival (token, OR, OR);
-		break;
-	}
-
-	if (combined != NULL) {
-		/* Inherit the location from the first token */
-		combined->location = token->location;
-		return combined;
-	}
-
-	/* Two string-valued (or integer) tokens can usually just be
-	 * mashed together. (We also handle a string followed by an
-	 * integer here as well.)
-	 *
-	 * There are some exceptions here. Notably, if the first token
-	 * is an integer (or a string representing an integer), then
-	 * the second token must also be an integer or must be a
-	 * string representing an integer that begins with a digit.
-	 */
-	if ((token->type == IDENTIFIER || token->type == OTHER || token->type == INTEGER_STRING || token->type == INTEGER) &&
-	    (other->type == IDENTIFIER || other->type == OTHER || other->type == INTEGER_STRING || other->type == INTEGER))
-	{
-		char *str;
-		int combined_type;
-
-		/* Check that pasting onto an integer doesn't create a
-		 * non-integer, (that is, only digits can be
-		 * pasted. */
-		if (token->type == INTEGER_STRING || token->type == INTEGER)
-		{
-			switch (other->type) {
-			case INTEGER_STRING:
-				if (other->value.str[0] < '0' ||
-				    other->value.str[0] > '9')
-					goto FAIL;
-				break;
-			case INTEGER:
-				if (other->value.ival < 0)
-					goto FAIL;
-				break;
-			default:
-				goto FAIL;
-			}
-		}
-
-		if (token->type == INTEGER)
-			str = ralloc_asprintf (token, "%" PRIiMAX,
-					       token->value.ival);
-		else
-			str = ralloc_strdup (token, token->value.str);
-					       
-
-		if (other->type == INTEGER)
-			ralloc_asprintf_append (&str, "%" PRIiMAX,
-						other->value.ival);
-		else
-			ralloc_strcat (&str, other->value.str);
-
-		/* New token is same type as original token, unless we
-		 * started with an integer, in which case we will be
-		 * creating an integer-string. */
-		combined_type = token->type;
-		if (combined_type == INTEGER)
-			combined_type = INTEGER_STRING;
-
-		combined = _token_create_str (token, combined_type, str);
-		combined->location = token->location;
-		return combined;
-	}
+   token_t *combined = NULL;
+
+   /* Pasting a placeholder onto anything makes no change. */
+   if (other->type == PLACEHOLDER)
+      return token;
+
+   /* When 'token' is a placeholder, just return 'other'. */
+   if (token->type == PLACEHOLDER)
+      return other;
+
+   /* A very few single-character punctuators can be combined
+    * with another to form a multi-character punctuator. */
+   switch (token->type) {
+   case '<':
+      if (other->type == '<')
+         combined = _token_create_ival (token, LEFT_SHIFT, LEFT_SHIFT);
+      else if (other->type == '=')
+         combined = _token_create_ival (token, LESS_OR_EQUAL, LESS_OR_EQUAL);
+      break;
+   case '>':
+      if (other->type == '>')
+         combined = _token_create_ival (token, RIGHT_SHIFT, RIGHT_SHIFT);
+      else if (other->type == '=')
+         combined = _token_create_ival (token, GREATER_OR_EQUAL, GREATER_OR_EQUAL);
+      break;
+   case '=':
+      if (other->type == '=')
+         combined = _token_create_ival (token, EQUAL, EQUAL);
+      break;
+   case '!':
+      if (other->type == '=')
+         combined = _token_create_ival (token, NOT_EQUAL, NOT_EQUAL);
+      break;
+   case '&':
+      if (other->type == '&')
+         combined = _token_create_ival (token, AND, AND);
+      break;
+   case '|':
+      if (other->type == '|')
+         combined = _token_create_ival (token, OR, OR);
+      break;
+   }
+
+   if (combined != NULL) {
+      /* Inherit the location from the first token */
+      combined->location = token->location;
+      return combined;
+   }
+
+   /* Two string-valued (or integer) tokens can usually just be
+    * mashed together. (We also handle a string followed by an
+    * integer here as well.)
+    *
+    * There are some exceptions here. Notably, if the first token
+    * is an integer (or a string representing an integer), then
+    * the second token must also be an integer or must be a
+    * string representing an integer that begins with a digit.
+    */
+   if ((token->type == IDENTIFIER || token->type == OTHER || token->type == INTEGER_STRING || token->type == INTEGER) &&
+       (other->type == IDENTIFIER || other->type == OTHER || other->type == INTEGER_STRING || other->type == INTEGER))
+   {
+      char *str;
+      int combined_type;
+
+      /* Check that pasting onto an integer doesn't create a
+       * non-integer, (that is, only digits can be
+       * pasted. */
+      if (token->type == INTEGER_STRING || token->type == INTEGER) {
+         switch (other->type) {
+         case INTEGER_STRING:
+            if (other->value.str[0] < '0' || other->value.str[0] > '9')
+               goto FAIL;
+            break;
+         case INTEGER:
+            if (other->value.ival < 0)
+               goto FAIL;
+            break;
+         default:
+            goto FAIL;
+         }
+      }
+
+      if (token->type == INTEGER)
+         str = ralloc_asprintf (token, "%" PRIiMAX, token->value.ival);
+      else
+         str = ralloc_strdup (token, token->value.str);
+
+      if (other->type == INTEGER)
+         ralloc_asprintf_append (&str, "%" PRIiMAX, other->value.ival);
+      else
+         ralloc_strcat (&str, other->value.str);
+
+      /* New token is same type as original token, unless we
+       * started with an integer, in which case we will be
+       * creating an integer-string. */
+      combined_type = token->type;
+      if (combined_type == INTEGER)
+         combined_type = INTEGER_STRING;
+
+      combined = _token_create_str (token, combined_type, str);
+      combined->location = token->location;
+      return combined;
+   }
 
     FAIL:
-	glcpp_error (&token->location, parser, "");
-	ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "Pasting \"");
-	_token_print (&parser->info_log, &parser->info_log_length, token);
-	ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "\" and \"");
-	_token_print (&parser->info_log, &parser->info_log_length, other);
-	ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "\" does not give a valid preprocessing token.\n");
-
-	return token;
+   glcpp_error (&token->location, parser, "");
+   ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "Pasting \"");
+   _token_print (&parser->info_log, &parser->info_log_length, token);
+   ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "\" and \"");
+   _token_print (&parser->info_log, &parser->info_log_length, other);
+   ralloc_asprintf_rewrite_tail (&parser->info_log, &parser->info_log_length, "\" does not give a valid preprocessing token.\n");
+
+   return token;
 }
 
 static void
-_token_list_print (glcpp_parser_t *parser, token_list_t *list)
+_token_list_print(glcpp_parser_t *parser, token_list_t *list)
 {
-	token_node_t *node;
+   token_node_t *node;
 
-	if (list == NULL)
-		return;
+   if (list == NULL)
+      return;
 
-	for (node = list->head; node; node = node->next)
-		_token_print (&parser->output, &parser->output_length, node->token);
+   for (node = list->head; node; node = node->next)
+      _token_print (&parser->output, &parser->output_length, node->token);
 }
 
 void
-yyerror (YYLTYPE *locp, glcpp_parser_t *parser, const char *error)
+yyerror(YYLTYPE *locp, glcpp_parser_t *parser, const char *error)
 {
-	glcpp_error(locp, parser, "%s", error);
+   glcpp_error(locp, parser, "%s", error);
 }
 
-static void add_builtin_define(glcpp_parser_t *parser,
-			       const char *name, int value)
+static void
+add_builtin_define(glcpp_parser_t *parser, const char *name, int value)
 {
    token_t *tok;
    token_list_t *list;
@@ -1325,63 +1311,63 @@ static void add_builtin_define(glcpp_parser_t *parser,
 }
 
 glcpp_parser_t *
-glcpp_parser_create (const struct gl_extensions *extensions, gl_api api)
+glcpp_parser_create(const struct gl_extensions *extensions, gl_api api)
 {
-	glcpp_parser_t *parser;
-
-	parser = ralloc (NULL, glcpp_parser_t);
-
-	glcpp_lex_init_extra (parser, &parser->scanner);
-	parser->defines = hash_table_ctor (32, hash_table_string_hash,
-					   hash_table_string_compare);
-	parser->active = NULL;
-	parser->lexing_directive = 0;
-	parser->space_tokens = 1;
-	parser->last_token_was_newline = 0;
-	parser->last_token_was_space = 0;
-	parser->first_non_space_token_this_line = 1;
-	parser->newline_as_space = 0;
-	parser->in_control_line = 0;
-	parser->paren_count = 0;
-        parser->commented_newlines = 0;
-
-	parser->skip_stack = NULL;
-	parser->skipping = 0;
-
-	parser->lex_from_list = NULL;
-	parser->lex_from_node = NULL;
-
-	parser->output = ralloc_strdup(parser, "");
-	parser->output_length = 0;
-	parser->info_log = ralloc_strdup(parser, "");
-	parser->info_log_length = 0;
-	parser->error = 0;
-
-        parser->extensions = extensions;
-        parser->api = api;
-        parser->version_resolved = false;
-
-	parser->has_new_line_number = 0;
-	parser->new_line_number = 1;
-	parser->has_new_source_number = 0;
-	parser->new_source_number = 0;
-
-	return parser;
+   glcpp_parser_t *parser;
+
+   parser = ralloc (NULL, glcpp_parser_t);
+
+   glcpp_lex_init_extra (parser, &parser->scanner);
+   parser->defines = hash_table_ctor(32, hash_table_string_hash,
+                                     hash_table_string_compare);
+   parser->active = NULL;
+   parser->lexing_directive = 0;
+   parser->space_tokens = 1;
+   parser->last_token_was_newline = 0;
+   parser->last_token_was_space = 0;
+   parser->first_non_space_token_this_line = 1;
+   parser->newline_as_space = 0;
+   parser->in_control_line = 0;
+   parser->paren_count = 0;
+   parser->commented_newlines = 0;
+
+   parser->skip_stack = NULL;
+   parser->skipping = 0;
+
+   parser->lex_from_list = NULL;
+   parser->lex_from_node = NULL;
+
+   parser->output = ralloc_strdup(parser, "");
+   parser->output_length = 0;
+   parser->info_log = ralloc_strdup(parser, "");
+   parser->info_log_length = 0;
+   parser->error = 0;
+
+   parser->extensions = extensions;
+   parser->api = api;
+   parser->version_resolved = false;
+
+   parser->has_new_line_number = 0;
+   parser->new_line_number = 1;
+   parser->has_new_source_number = 0;
+   parser->new_source_number = 0;
+
+   return parser;
 }
 
 void
-glcpp_parser_destroy (glcpp_parser_t *parser)
+glcpp_parser_destroy(glcpp_parser_t *parser)
 {
-	glcpp_lex_destroy (parser->scanner);
-	hash_table_dtor (parser->defines);
-	ralloc_free (parser);
+   glcpp_lex_destroy (parser->scanner);
+   hash_table_dtor (parser->defines);
+   ralloc_free (parser);
 }
 
 typedef enum function_status
 {
-	FUNCTION_STATUS_SUCCESS,
-	FUNCTION_NOT_A_FUNCTION,
-	FUNCTION_UNBALANCED_PARENTHESES
+   FUNCTION_STATUS_SUCCESS,
+   FUNCTION_NOT_A_FUNCTION,
+   FUNCTION_UNBALANCED_PARENTHESES
 } function_status_t;
 
 /* Find a set of function-like macro arguments by looking for a
@@ -1396,100 +1382,92 @@ typedef enum function_status
  *
  *   FUNCTION_STATUS_SUCCESS:
  *
- *	Successfully parsed a set of function arguments.	
+ *      Successfully parsed a set of function arguments.
  *
  *   FUNCTION_NOT_A_FUNCTION:
  *
- *	Macro name not followed by a '('. This is not an error, but
- *	simply that the macro name should be treated as a non-macro.
+ *      Macro name not followed by a '('. This is not an error, but
+ *      simply that the macro name should be treated as a non-macro.
  *
  *   FUNCTION_UNBALANCED_PARENTHESES
  *
- *	Macro name is not followed by a balanced set of parentheses.
+ *      Macro name is not followed by a balanced set of parentheses.
  */
 static function_status_t
-_arguments_parse (argument_list_t *arguments,
-		  token_node_t *node,
-		  token_node_t **last)
+_arguments_parse(argument_list_t *arguments, token_node_t *node,
+                 token_node_t **last)
 {
-	token_list_t *argument;
-	int paren_count;
-
-	node = node->next;
-
-	/* Ignore whitespace before first parenthesis. */
-	while (node && node->token->type == SPACE)
-		node = node->next;
-
-	if (node == NULL || node->token->type != '(')
-		return FUNCTION_NOT_A_FUNCTION;
-
-	node = node->next;
-
-	argument = _token_list_create (arguments);
-	_argument_list_append (arguments, argument);
-
-	for (paren_count = 1; node; node = node->next) {
-		if (node->token->type == '(')
-		{
-			paren_count++;
-		}
-		else if (node->token->type == ')')
-		{
-			paren_count--;
-			if (paren_count == 0)
-				break;
-		}
-
-		if (node->token->type == ',' &&
-			 paren_count == 1)
-		{
-			_token_list_trim_trailing_space (argument);
-			argument = _token_list_create (arguments);
-			_argument_list_append (arguments, argument);
-		}
-		else {
-			if (argument->head == NULL) {
-				/* Don't treat initial whitespace as
-				 * part of the argument. */
-				if (node->token->type == SPACE)
-					continue;
-			}
-			_token_list_append (argument, node->token);
-		}
-	}
-
-	if (paren_count)
-		return FUNCTION_UNBALANCED_PARENTHESES;
-
-	*last = node;
-
-	return FUNCTION_STATUS_SUCCESS;
+   token_list_t *argument;
+   int paren_count;
+
+   node = node->next;
+
+   /* Ignore whitespace before first parenthesis. */
+   while (node && node->token->type == SPACE)
+      node = node->next;
+
+   if (node == NULL || node->token->type != '(')
+      return FUNCTION_NOT_A_FUNCTION;
+
+   node = node->next;
+
+   argument = _token_list_create (arguments);
+   _argument_list_append (arguments, argument);
+
+   for (paren_count = 1; node; node = node->next) {
+      if (node->token->type == '(') {
+         paren_count++;
+      } else if (node->token->type == ')') {
+         paren_count--;
+         if (paren_count == 0)
+            break;
+      }
+
+      if (node->token->type == ',' && paren_count == 1) {
+         _token_list_trim_trailing_space (argument);
+         argument = _token_list_create (arguments);
+         _argument_list_append (arguments, argument);
+      } else {
+         if (argument->head == NULL) {
+            /* Don't treat initial whitespace as part of the argument. */
+            if (node->token->type == SPACE)
+               continue;
+         }
+         _token_list_append (argument, node->token);
+      }
+   }
+
+   if (paren_count)
+      return FUNCTION_UNBALANCED_PARENTHESES;
+
+   *last = node;
+
+   return FUNCTION_STATUS_SUCCESS;
 }
 
 static token_list_t *
-_token_list_create_with_one_ival (void *ctx, int type, int ival)
+_token_list_create_with_one_ival(void *ctx, int type, int ival)
 {
-	token_list_t *list;
-	token_t *node;
+   token_list_t *list;
+   token_t *node;
 
-	list = _token_list_create (ctx);
-	node = _token_create_ival (list, type, ival);
-	_token_list_append (list, node);
+   list = _token_list_create(ctx);
+   node = _token_create_ival(list, type, ival);
+   _token_list_append(list, node);
 
-	return list;
+   return list;
 }
 
 static token_list_t *
-_token_list_create_with_one_space (void *ctx)
+_token_list_create_with_one_space(void *ctx)
 {
-	return _token_list_create_with_one_ival (ctx, SPACE, SPACE);
+   return _token_list_create_with_one_ival(ctx, SPACE, SPACE);
 }
 
 static token_list_t *
-_token_list_create_with_one_integer (void *ctx, int ival)
+_token_list_create_with_one_integer(void *ctx, int ival)
 {
-	return _token_list_create_with_one_ival (ctx, INTEGER, ival);
+   return _token_list_create_with_one_ival(ctx, INTEGER, ival);
 }
 
 /* Evaluate a DEFINED token node (based on subsequent tokens in the list).
@@ -1501,119 +1479,117 @@ _token_list_create_with_one_integer (void *ctx, int ival)
  * (optionally preceded and followed by '(' and ')' tokens) then the following
  * occurs:
  *
- *	If the identifier is a defined macro, this function returns 1.
+ *   If the identifier is a defined macro, this function returns 1.
  *
- *	If the identifier is not a defined macro, this function returns 0.
+ *   If the identifier is not a defined macro, this function returns 0.
  *
- *	In either case, *last will be updated to the last node in the list
- *	consumed by the evaluation, (either the token of the identifier or the
- *	token of the closing parenthesis).
+ *   In either case, *last will be updated to the last node in the list
+ *   consumed by the evaluation, (either the token of the identifier or the
+ *   token of the closing parenthesis).
  *
  * In all other cases, (such as "node is the final node of the list", or
  * "missing closing parenthesis", etc.), this function generates a
  * preprocessor error, returns -1 and *last will not be set.
  */
 static int
-_glcpp_parser_evaluate_defined (glcpp_parser_t *parser,
-				token_node_t *node,
-				token_node_t **last)
+_glcpp_parser_evaluate_defined(glcpp_parser_t *parser, token_node_t *node,
+                               token_node_t **last)
 {
-	token_node_t *argument, *defined = node;
+   token_node_t *argument, *defined = node;
 
-	assert (node->token->type == DEFINED);
+   assert(node->token->type == DEFINED);
 
-	node = node->next;
+   node = node->next;
 
-	/* Ignore whitespace after DEFINED token. */
-	while (node && node->token->type == SPACE)
-		node = node->next;
+   /* Ignore whitespace after DEFINED token. */
+   while (node && node->token->type == SPACE)
+      node = node->next;
 
-	if (node == NULL)
-		goto FAIL;
+   if (node == NULL)
+      goto FAIL;
 
-	if (node->token->type == IDENTIFIER || node->token->type == OTHER) {
-		argument = node;
-	} else if (node->token->type == '(') {
-		node = node->next;
+   if (node->token->type == IDENTIFIER || node->token->type == OTHER) {
+      argument = node;
+   } else if (node->token->type == '(') {
+      node = node->next;
 
-		/* Ignore whitespace after '(' token. */
-		while (node && node->token->type == SPACE)
-			node = node->next;
+      /* Ignore whitespace after '(' token. */
+      while (node && node->token->type == SPACE)
+         node = node->next;
 
-		if (node == NULL || (node->token->type != IDENTIFIER &&
-				     node->token->type != OTHER))
-		{
-			goto FAIL;
-		}
+      if (node == NULL || (node->token->type != IDENTIFIER &&
+                           node->token->type != OTHER)) {
+         goto FAIL;
+      }
 
-		argument = node;
+      argument = node;
 
-		node = node->next;
+      node = node->next;
 
-		/* Ignore whitespace after identifier, before ')' token. */
-		while (node && node->token->type == SPACE)
-			node = node->next;
+      /* Ignore whitespace after identifier, before ')' token. */
+      while (node && node->token->type == SPACE)
+         node = node->next;
 
-		if (node == NULL || node->token->type != ')')
-			goto FAIL;
-	} else {
-		goto FAIL;
-	}
+      if (node == NULL || node->token->type != ')')
+         goto FAIL;
+   } else {
+      goto FAIL;
+   }
 
-	*last = node;
+   *last = node;
 
-	return hash_table_find (parser->defines,
-				argument->token->value.str) ? 1 : 0;
+   return hash_table_find(parser->defines,
+                          argument->token->value.str) ? 1 : 0;
 
 FAIL:
-	glcpp_error (&defined->token->location, parser,
-		     "\"defined\" not followed by an identifier");
-	return -1;
+   glcpp_error (&defined->token->location, parser,
+                "\"defined\" not followed by an identifier");
+   return -1;
 }
 
 /* Evaluate all DEFINED nodes in a given list, modifying the list in place.
  */
 static void
-_glcpp_parser_evaluate_defined_in_list (glcpp_parser_t *parser,
-					token_list_t *list)
+_glcpp_parser_evaluate_defined_in_list(glcpp_parser_t *parser,
+                                       token_list_t *list)
 {
-	token_node_t *node, *node_prev, *replacement, *last = NULL;
-	int value;
+   token_node_t *node, *node_prev, *replacement, *last = NULL;
+   int value;
 
-	if (list == NULL)
-		return;
+   if (list == NULL)
+      return;
 
-	node_prev = NULL;
-	node = list->head;
+   node_prev = NULL;
+   node = list->head;
 
-	while (node) {
+   while (node) {
 
-		if (node->token->type != DEFINED)
-			goto NEXT;
+      if (node->token->type != DEFINED)
+         goto NEXT;
 
-		value = _glcpp_parser_evaluate_defined (parser, node, &last);
-		if (value == -1)
-			goto NEXT;
+      value = _glcpp_parser_evaluate_defined (parser, node, &last);
+      if (value == -1)
+         goto NEXT;
 
-		replacement = ralloc (list, token_node_t);
-		replacement->token = _token_create_ival (list, INTEGER, value);
+      replacement = ralloc (list, token_node_t);
+      replacement->token = _token_create_ival (list, INTEGER, value);
 
-		/* Splice replacement node into list, replacing from "node"
-		 * through "last". */
-		if (node_prev)
-			node_prev->next = replacement;
-		else
-			list->head = replacement;
-		replacement->next = last->next;
-		if (last == list->tail)
-			list->tail = replacement;
+      /* Splice replacement node into list, replacing from "node"
+       * through "last". */
+      if (node_prev)
+         node_prev->next = replacement;
+      else
+         list->head = replacement;
+      replacement->next = last->next;
+      if (last == list->tail)
+         list->tail = replacement;
 
-		node = replacement;
+      node = replacement;
 
-	NEXT:
-		node_prev = node;
-		node = node->next;
-	}
+   NEXT:
+      node_prev = node;
+      node = node->next;
+   }
 }
 
 /* Perform macro expansion on 'list', placing the resulting tokens
@@ -1625,62 +1601,59 @@ _glcpp_parser_evaluate_defined_in_list (glcpp_parser_t *parser,
  * of the "mode" parameter.
  */
 static void
-_glcpp_parser_expand_and_lex_from (glcpp_parser_t *parser,
-				   int head_token_type,
-				   token_list_t *list,
-				   expansion_mode_t mode)
+_glcpp_parser_expand_and_lex_from(glcpp_parser_t *parser, int head_token_type,
+                                  token_list_t *list, expansion_mode_t mode)
 {
-	token_list_t *expanded;
-	token_t *token;
-
-	expanded = _token_list_create (parser);
-	token = _token_create_ival (parser, head_token_type, head_token_type);
-	_token_list_append (expanded, token);
-	_glcpp_parser_expand_token_list (parser, list, mode);
-	_token_list_append_list (expanded, list);
-	glcpp_parser_lex_from (parser, expanded);
+   token_list_t *expanded;
+   token_t *token;
+
+   expanded = _token_list_create (parser);
+   token = _token_create_ival (parser, head_token_type, head_token_type);
+   _token_list_append (expanded, token);
+   _glcpp_parser_expand_token_list (parser, list, mode);
+   _token_list_append_list (expanded, list);
+   glcpp_parser_lex_from (parser, expanded);
 }
 
 static void
-_glcpp_parser_apply_pastes (glcpp_parser_t *parser, token_list_t *list)
+_glcpp_parser_apply_pastes(glcpp_parser_t *parser, token_list_t *list)
 {
-	token_node_t *node;
-
-	node = list->head;
-	while (node)
-	{
-		token_node_t *next_non_space;
-
-		/* Look ahead for a PASTE token, skipping space. */
-		next_non_space = node->next;
-		while (next_non_space && next_non_space->token->type == SPACE)
-			next_non_space = next_non_space->next;
-
-		if (next_non_space == NULL)
-			break;
-
-		if (next_non_space->token->type != PASTE) {
-			node = next_non_space;
-			continue;
-		}
-
-		/* Now find the next non-space token after the PASTE. */
-		next_non_space = next_non_space->next;
-		while (next_non_space && next_non_space->token->type == SPACE)
-			next_non_space = next_non_space->next;
-
-		if (next_non_space == NULL) {
-			yyerror (&node->token->location, parser, "'##' cannot appear at either end of a macro expansion\n");
-			return;
-		}
-
-		node->token = _token_paste (parser, node->token, next_non_space->token);
-		node->next = next_non_space->next;
-		if (next_non_space == list->tail)
-			list->tail = node;
-	}
-
-	list->non_space_tail = list->tail;
+   token_node_t *node;
+
+   node = list->head;
+   while (node) {
+      token_node_t *next_non_space;
+
+      /* Look ahead for a PASTE token, skipping space. */
+      next_non_space = node->next;
+      while (next_non_space && next_non_space->token->type == SPACE)
+         next_non_space = next_non_space->next;
+
+      if (next_non_space == NULL)
+         break;
+
+      if (next_non_space->token->type != PASTE) {
+         node = next_non_space;
+         continue;
+      }
+
+      /* Now find the next non-space token after the PASTE. */
+      next_non_space = next_non_space->next;
+      while (next_non_space && next_non_space->token->type == SPACE)
+         next_non_space = next_non_space->next;
+
+      if (next_non_space == NULL) {
+         yyerror(&node->token->location, parser, "'##' cannot appear at either end of a macro expansion\n");
+         return;
+      }
+
+      node->token = _token_paste(parser, node->token, next_non_space->token);
+      node->next = next_non_space->next;
+      if (next_non_space == list->tail)
+         list->tail = node;
+   }
+
+   list->non_space_tail = list->tail;
 }
 
 /* This is a helper function that's essentially part of the
@@ -1704,103 +1677,89 @@ _glcpp_parser_apply_pastes (glcpp_parser_t *parser, token_list_t *list)
  * of the "mode" parameter.
  */
 static token_list_t *
-_glcpp_parser_expand_function (glcpp_parser_t *parser,
-			       token_node_t *node,
-			       token_node_t **last,
-			       expansion_mode_t mode)
+_glcpp_parser_expand_function(glcpp_parser_t *parser, token_node_t *node,
+                              token_node_t **last, expansion_mode_t mode)
 {
-	macro_t *macro;
-	const char *identifier;
-	argument_list_t *arguments;
-	function_status_t status;
-	token_list_t *substituted;
-	int parameter_index;
-
-	identifier = node->token->value.str;
-
-	macro = hash_table_find (parser->defines, identifier);
-
-	assert (macro->is_function);
-
-	arguments = _argument_list_create (parser);
-	status = _arguments_parse (arguments, node, last);
-
-	switch (status) {
-	case FUNCTION_STATUS_SUCCESS:
-		break;
-	case FUNCTION_NOT_A_FUNCTION:
-		return NULL;
-	case FUNCTION_UNBALANCED_PARENTHESES:
-		glcpp_error (&node->token->location, parser, "Macro %s call has unbalanced parentheses\n", identifier);
-		return NULL;
-	}
-
-	/* Replace a macro defined as empty with a SPACE token. */
-	if (macro->replacements == NULL) {
-		ralloc_free (arguments);
-		return _token_list_create_with_one_space (parser);
-	}
-
-	if (! ((_argument_list_length (arguments) == 
-		_string_list_length (macro->parameters)) ||
-	       (_string_list_length (macro->parameters) == 0 &&
-		_argument_list_length (arguments) == 1 &&
-		arguments->head->argument->head == NULL)))
-	{
-		glcpp_error (&node->token->location, parser,
-			      "Error: macro %s invoked with %d arguments (expected %d)\n",
-			      identifier,
-			      _argument_list_length (arguments),
-			      _string_list_length (macro->parameters));
-		return NULL;
-	}
-
-	/* Perform argument substitution on the replacement list. */
-	substituted = _token_list_create (arguments);
-
-	for (node = macro->replacements->head; node; node = node->next)
-	{
-		if (node->token->type == IDENTIFIER &&
-		    _string_list_contains (macro->parameters,
-					   node->token->value.str,
-					   &parameter_index))
-		{
-			token_list_t *argument;
-			argument = _argument_list_member_at (arguments,
-							     parameter_index);
-			/* Before substituting, we expand the argument
-			 * tokens, or append a placeholder token for
-			 * an empty argument. */
-			if (argument->head) {
-				token_list_t *expanded_argument;
-				expanded_argument = _token_list_copy (parser,
-								      argument);
-				_glcpp_parser_expand_token_list (parser,
-								 expanded_argument,
-								 mode);
-				_token_list_append_list (substituted,
-							 expanded_argument);
-			} else {
-				token_t *new_token;
-
-				new_token = _token_create_ival (substituted,
-								PLACEHOLDER,
-								PLACEHOLDER);
-				_token_list_append (substituted, new_token);
-			}
-		} else {
-			_token_list_append (substituted, node->token);
-		}
-	}
-
-	/* After argument substitution, and before further expansion
-	 * below, implement token pasting. */
-
-	_token_list_trim_trailing_space (substituted);
-
-	_glcpp_parser_apply_pastes (parser, substituted);
-
-	return substituted;
+   macro_t *macro;
+   const char *identifier;
+   argument_list_t *arguments;
+   function_status_t status;
+   token_list_t *substituted;
+   int parameter_index;
+
+   identifier = node->token->value.str;
+
+   macro = hash_table_find(parser->defines, identifier);
+
+   assert(macro->is_function);
+
+   arguments = _argument_list_create(parser);
+   status = _arguments_parse(arguments, node, last);
+
+   switch (status) {
+   case FUNCTION_STATUS_SUCCESS:
+      break;
+   case FUNCTION_NOT_A_FUNCTION:
+      return NULL;
+   case FUNCTION_UNBALANCED_PARENTHESES:
+      glcpp_error(&node->token->location, parser, "Macro %s call has unbalanced parentheses\n", identifier);
+      return NULL;
+   }
+
+   /* Replace a macro defined as empty with a SPACE token. */
+   if (macro->replacements == NULL) {
+      ralloc_free(arguments);
+      return _token_list_create_with_one_space(parser);
+   }
+
+   if (!((_argument_list_length (arguments) == 
+          _string_list_length (macro->parameters)) ||
+         (_string_list_length (macro->parameters) == 0 &&
+          _argument_list_length (arguments) == 1 &&
+          arguments->head->argument->head == NULL))) {
+      glcpp_error(&node->token->location, parser,
+                  "Error: macro %s invoked with %d arguments (expected %d)\n",
+                  identifier, _argument_list_length (arguments),
+                  _string_list_length(macro->parameters));
+      return NULL;
+   }
+
+   /* Perform argument substitution on the replacement list. */
+   substituted = _token_list_create(arguments);
+
+   for (node = macro->replacements->head; node; node = node->next) {
+      if (node->token->type == IDENTIFIER &&
+          _string_list_contains(macro->parameters, node->token->value.str,
+                                &parameter_index)) {
+         token_list_t *argument;
+         argument = _argument_list_member_at(arguments, parameter_index);
+         /* Before substituting, we expand the argument tokens, or append a
+          * placeholder token for an empty argument. */
+         if (argument->head) {
+            token_list_t *expanded_argument;
+            expanded_argument = _token_list_copy(parser, argument);
+            _glcpp_parser_expand_token_list(parser, expanded_argument, mode);
+            _token_list_append_list(substituted, expanded_argument);
+         } else {
+            token_t *new_token;
+
+            new_token = _token_create_ival(substituted, PLACEHOLDER,
+                                           PLACEHOLDER);
+            _token_list_append(substituted, new_token);
+         }
+      } else {
+         _token_list_append(substituted, node->token);
+      }
+   }
+
+   /* After argument substitution, and before further expansion
+    * below, implement token pasting. */
+
+   _token_list_trim_trailing_space(substituted);
+
+   _glcpp_parser_apply_pastes(parser, substituted);
+
+   return substituted;
 }
 
 /* Compute the complete expansion of node, (and subsequent nodes after
@@ -1813,78 +1772,74 @@ _glcpp_parser_expand_function (glcpp_parser_t *parser,
  * and sets *last to the last node in the list that was consumed by
  * the expansion. Specifically, *last will be set as follows:
  *
- *	As 'node' in the case of object-like macro expansion.
+ *   As 'node' in the case of object-like macro expansion.
  *
- *	As the token of the closing right parenthesis in the case of
- *	function-like macro expansion.
+ *   As the token of the closing right parenthesis in the case of
+ *   function-like macro expansion.
  *
  * See the documentation of _glcpp_parser_expand_token_list for a description
  * of the "mode" parameter.
  */
 static token_list_t *
-_glcpp_parser_expand_node (glcpp_parser_t *parser,
-			   token_node_t *node,
-			   token_node_t **last,
-			   expansion_mode_t mode)
+_glcpp_parser_expand_node(glcpp_parser_t *parser, token_node_t *node,
+                          token_node_t **last, expansion_mode_t mode)
 {
-	token_t *token = node->token;
-	const char *identifier;
-	macro_t *macro;
-
-	/* We only expand identifiers */
-	if (token->type != IDENTIFIER) {
-		return NULL;
-	}
-
-	*last = node;
-	identifier = token->value.str;
-
-	/* Special handling for __LINE__ and __FILE__, (not through
-	 * the hash table). */
-	if (strcmp(identifier, "__LINE__") == 0)
-		return _token_list_create_with_one_integer (parser, node->token->location.first_line);
-
-	if (strcmp(identifier, "__FILE__") == 0)
-		return _token_list_create_with_one_integer (parser, node->token->location.source);
-
-	/* Look up this identifier in the hash table. */
-	macro = hash_table_find (parser->defines, identifier);
-
-	/* Not a macro, so no expansion needed. */
-	if (macro == NULL)
-		return NULL;
-
-	/* Finally, don't expand this macro if we're already actively
-	 * expanding it, (to avoid infinite recursion). */
-	if (_parser_active_list_contains (parser, identifier)) {
-		/* We change the token type here from IDENTIFIER to
-		 * OTHER to prevent any future expansion of this
-		 * unexpanded token. */
-		char *str;
-		token_list_t *expansion;
-		token_t *final;
-
-		str = ralloc_strdup (parser, token->value.str);
-		final = _token_create_str (parser, OTHER, str);
-		expansion = _token_list_create (parser);
-		_token_list_append (expansion, final);
-		return expansion;
-	}
-
-	if (! macro->is_function)
-	{
-		token_list_t *replacement;
-
-		/* Replace a macro defined as empty with a SPACE token. */
-		if (macro->replacements == NULL)
-			return _token_list_create_with_one_space (parser);
-
-		replacement = _token_list_copy (parser, macro->replacements);
-		_glcpp_parser_apply_pastes (parser, replacement);
-		return replacement;
-	}
-
-	return _glcpp_parser_expand_function (parser, node, last, mode);
+   token_t *token = node->token;
+   const char *identifier;
+   macro_t *macro;
+
+   /* We only expand identifiers */
+   if (token->type != IDENTIFIER) {
+      return NULL;
+   }
+
+   *last = node;
+   identifier = token->value.str;
+
+   /* Special handling for __LINE__ and __FILE__, (not through
+    * the hash table). */
+   if (strcmp(identifier, "__LINE__") == 0)
+      return _token_list_create_with_one_integer(parser, node->token->location.first_line);
+
+   if (strcmp(identifier, "__FILE__") == 0)
+      return _token_list_create_with_one_integer(parser, node->token->location.source);
+
+   /* Look up this identifier in the hash table. */
+   macro = hash_table_find(parser->defines, identifier);
+
+   /* Not a macro, so no expansion needed. */
+   if (macro == NULL)
+      return NULL;
+
+   /* Finally, don't expand this macro if we're already actively
+    * expanding it, (to avoid infinite recursion). */
+   if (_parser_active_list_contains (parser, identifier)) {
+      /* We change the token type here from IDENTIFIER to OTHER to prevent any
+       * future expansion of this unexpanded token. */
+      char *str;
+      token_list_t *expansion;
+      token_t *final;
+
+      str = ralloc_strdup(parser, token->value.str);
+      final = _token_create_str(parser, OTHER, str);
+      expansion = _token_list_create(parser);
+      _token_list_append(expansion, final);
+      return expansion;
+   }
+
+   if (! macro->is_function) {
+      token_list_t *replacement;
+
+      /* Replace a macro defined as empty with a SPACE token. */
+      if (macro->replacements == NULL)
+         return _token_list_create_with_one_space(parser);
+
+      replacement = _token_list_copy(parser, macro->replacements);
+      _glcpp_parser_apply_pastes(parser, replacement);
+      return replacement;
+   }
+
+   return _glcpp_parser_expand_function(parser, node, last, mode);
 }
 
 /* Push a new identifier onto the parser's active list.
@@ -1895,49 +1850,48 @@ _glcpp_parser_expand_node (glcpp_parser_t *parser,
  * active stack.
  */
 static void
-_parser_active_list_push (glcpp_parser_t *parser,
-			  const char *identifier,
-			  token_node_t *marker)
+_parser_active_list_push(glcpp_parser_t *parser, const char *identifier,
+                         token_node_t *marker)
 {
-	active_list_t *node;
+   active_list_t *node;
 
-	node = ralloc (parser->active, active_list_t);
-	node->identifier = ralloc_strdup (node, identifier);
-	node->marker = marker;
-	node->next = parser->active;
+   node = ralloc(parser->active, active_list_t);
+   node->identifier = ralloc_strdup(node, identifier);
+   node->marker = marker;
+   node->next = parser->active;
 
-	parser->active = node;
+   parser->active = node;
 }
 
 static void
-_parser_active_list_pop (glcpp_parser_t *parser)
+_parser_active_list_pop(glcpp_parser_t *parser)
 {
-	active_list_t *node = parser->active;
+   active_list_t *node = parser->active;
 
-	if (node == NULL) {
-		parser->active = NULL;
-		return;
-	}
+   if (node == NULL) {
+      parser->active = NULL;
+      return;
+   }
 
-	node = parser->active->next;
-	ralloc_free (parser->active);
+   node = parser->active->next;
+   ralloc_free (parser->active);
 
-	parser->active = node;
+   parser->active = node;
 }
 
 static int
-_parser_active_list_contains (glcpp_parser_t *parser, const char *identifier)
+_parser_active_list_contains(glcpp_parser_t *parser, const char *identifier)
 {
-	active_list_t *node;
+   active_list_t *node;
 
-	if (parser->active == NULL)
-		return 0;
+   if (parser->active == NULL)
+      return 0;
 
-	for (node = parser->active; node; node = node->next)
-		if (strcmp (node->identifier, identifier) == 0)
-			return 1;
+   for (node = parser->active; node; node = node->next)
+      if (strcmp(node->identifier, identifier) == 0)
+         return 1;
 
-	return 0;
+   return 0;
 }
 
 /* Walk over the token list replacing nodes with their expansion.
@@ -1948,400 +1902,376 @@ _parser_active_list_contains (glcpp_parser_t *parser, const char *identifier)
  * The "mode" argument controls the handling of any DEFINED tokens that
  * result from expansion as follows:
  *
- *	EXPANSION_MODE_IGNORE_DEFINED: Any resulting DEFINED tokens will be
- *		left in the final list, unevaluated. This is the correct mode
- *		for expanding any list in any context other than a
- *		preprocessor conditional, (#if or #elif).
+ *   EXPANSION_MODE_IGNORE_DEFINED: Any resulting DEFINED tokens will be
+ *      left in the final list, unevaluated. This is the correct mode
+ *      for expanding any list in any context other than a
+ *      preprocessor conditional, (#if or #elif).
  *
- *	EXPANSION_MODE_EVALUATE_DEFINED: Any resulting DEFINED tokens will be
- *		evaluated to 0 or 1 tokens depending on whether the following
- *		token is the name of a defined macro. If the DEFINED token is
- *		not followed by an (optionally parenthesized) identifier, then
- *		an error will be generated. This the correct mode for
- *		expanding any list in the context of a preprocessor
- *		conditional, (#if or #elif).
+ *   EXPANSION_MODE_EVALUATE_DEFINED: Any resulting DEFINED tokens will be
+ *      evaluated to 0 or 1 tokens depending on whether the following
+ *      token is the name of a defined macro. If the DEFINED token is
+ *      not followed by an (optionally parenthesized) identifier, then
+ *      an error will be generated. This the correct mode for
+ *      expanding any list in the context of a preprocessor
+ *      conditional, (#if or #elif).
  */
 static void
-_glcpp_parser_expand_token_list (glcpp_parser_t *parser,
-				 token_list_t *list,
-				 expansion_mode_t mode)
+_glcpp_parser_expand_token_list(glcpp_parser_t *parser, token_list_t *list,
+                                expansion_mode_t mode)
 {
-	token_node_t *node_prev;
-	token_node_t *node, *last = NULL;
-	token_list_t *expansion;
-	active_list_t *active_initial = parser->active;
-
-	if (list == NULL)
-		return;
-
-	_token_list_trim_trailing_space (list);
-
-	node_prev = NULL;
-	node = list->head;
-
-	if (mode == EXPANSION_MODE_EVALUATE_DEFINED)
-		_glcpp_parser_evaluate_defined_in_list (parser, list);
-
-	while (node) {
-
-		while (parser->active && parser->active->marker == node)
-			_parser_active_list_pop (parser);
-
-		expansion = _glcpp_parser_expand_node (parser, node, &last, mode);
-		if (expansion) {
-			token_node_t *n;
-
-			if (mode == EXPANSION_MODE_EVALUATE_DEFINED) {
-				_glcpp_parser_evaluate_defined_in_list (parser,
-									expansion);
-			}
-
-			for (n = node; n != last->next; n = n->next)
-				while (parser->active &&
-				       parser->active->marker == n)
-				{
-					_parser_active_list_pop (parser);
-				}
-
-			_parser_active_list_push (parser,
-						  node->token->value.str,
-						  last->next);
-			
-			/* Splice expansion into list, supporting a
-			 * simple deletion if the expansion is
-			 * empty. */
-			if (expansion->head) {
-				if (node_prev)
-					node_prev->next = expansion->head;
-				else
-					list->head = expansion->head;
-				expansion->tail->next = last->next;
-				if (last == list->tail)
-					list->tail = expansion->tail;
-			} else {
-				if (node_prev)
-					node_prev->next = last->next;
-				else
-					list->head = last->next;
-				if (last == list->tail)
-					list->tail = NULL;
-			}
-		} else {
-			node_prev = node;
-		}
-		node = node_prev ? node_prev->next : list->head;
-	}
-
-	/* Remove any lingering effects of this invocation on the
-	 * active list. That is, pop until the list looks like it did
-	 * at the beginning of this function. */
-	while (parser->active && parser->active != active_initial)
-		_parser_active_list_pop (parser);
-
-	list->non_space_tail = list->tail;
+   token_node_t *node_prev;
+   token_node_t *node, *last = NULL;
+   token_list_t *expansion;
+   active_list_t *active_initial = parser->active;
+
+   if (list == NULL)
+      return;
+
+   _token_list_trim_trailing_space (list);
+
+   node_prev = NULL;
+   node = list->head;
+
+   if (mode == EXPANSION_MODE_EVALUATE_DEFINED)
+      _glcpp_parser_evaluate_defined_in_list (parser, list);
+
+   while (node) {
+
+      while (parser->active && parser->active->marker == node)
+         _parser_active_list_pop (parser);
+
+      expansion = _glcpp_parser_expand_node (parser, node, &last, mode);
+      if (expansion) {
+         token_node_t *n;
+
+         if (mode == EXPANSION_MODE_EVALUATE_DEFINED) {
+            _glcpp_parser_evaluate_defined_in_list (parser, expansion);
+         }
+
+         for (n = node; n != last->next; n = n->next)
+            while (parser->active && parser->active->marker == n) {
+               _parser_active_list_pop (parser);
+            }
+
+         _parser_active_list_push(parser, node->token->value.str, last->next);
+         
+         /* Splice expansion into list, supporting a simple deletion if the
+          * expansion is empty.
+          */
+         if (expansion->head) {
+            if (node_prev)
+               node_prev->next = expansion->head;
+            else
+               list->head = expansion->head;
+            expansion->tail->next = last->next;
+            if (last == list->tail)
+               list->tail = expansion->tail;
+         } else {
+            if (node_prev)
+               node_prev->next = last->next;
+            else
+               list->head = last->next;
+            if (last == list->tail)
+               list->tail = NULL;
+         }
+      } else {
+         node_prev = node;
+      }
+      node = node_prev ? node_prev->next : list->head;
+   }
+
+   /* Remove any lingering effects of this invocation on the
+    * active list. That is, pop until the list looks like it did
+    * at the beginning of this function. */
+   while (parser->active && parser->active != active_initial)
+      _parser_active_list_pop (parser);
+
+   list->non_space_tail = list->tail;
 }
 
 void
-_glcpp_parser_print_expanded_token_list (glcpp_parser_t *parser,
-					 token_list_t *list)
+_glcpp_parser_print_expanded_token_list(glcpp_parser_t *parser,
+                                        token_list_t *list)
 {
-	if (list == NULL)
-		return;
+   if (list == NULL)
+      return;
 
-	_glcpp_parser_expand_token_list (parser, list, EXPANSION_MODE_IGNORE_DEFINED);
+   _glcpp_parser_expand_token_list (parser, list, EXPANSION_MODE_IGNORE_DEFINED);
 
-	_token_list_trim_trailing_space (list);
+   _token_list_trim_trailing_space (list);
 
-	_token_list_print (parser, list);
+   _token_list_print (parser, list);
 }
 
 static void
-_check_for_reserved_macro_name (glcpp_parser_t *parser, YYLTYPE *loc,
-				const char *identifier)
+_check_for_reserved_macro_name(glcpp_parser_t *parser, YYLTYPE *loc,
+                               const char *identifier)
 {
-	/* Section 3.3 (Preprocessor) of the GLSL 1.30 spec (and later) and
-	 * the GLSL ES spec (all versions) say:
-	 *
-	 *     "All macro names containing two consecutive underscores ( __ )
-	 *     are reserved for future use as predefined macro names. All
-	 *     macro names prefixed with "GL_" ("GL" followed by a single
-	 *     underscore) are also reserved."
-	 *
-	 * The intention is that names containing __ are reserved for internal
-	 * use by the implementation, and names prefixed with GL_ are reserved
-	 * for use by Khronos.  Since every extension adds a name prefixed
-	 * with GL_ (i.e., the name of the extension), that should be an
-	 * error.  Names simply containing __ are dangerous to use, but should
-	 * be allowed.
-	 *
-	 * A future version of the GLSL specification will clarify this.
-	 */
-	if (strstr(identifier, "__")) {
-		glcpp_warning(loc, parser,
-			      "Macro names containing \"__\" are reserved "
-			      "for use by the implementation.\n");
-	}
-	if (strncmp(identifier, "GL_", 3) == 0) {
-		glcpp_error (loc, parser, "Macro names starting with \"GL_\" are reserved.\n");
-	}
-	if (strcmp(identifier, "defined") == 0) {
-		glcpp_error (loc, parser, "\"defined\" cannot be used as a macro name");
-	}
+   /* Section 3.3 (Preprocessor) of the GLSL 1.30 spec (and later) and
+    * the GLSL ES spec (all versions) say:
+    *
+    *     "All macro names containing two consecutive underscores ( __ )
+    *     are reserved for future use as predefined macro names. All
+    *     macro names prefixed with "GL_" ("GL" followed by a single
+    *     underscore) are also reserved."
+    *
+    * The intention is that names containing __ are reserved for internal
+    * use by the implementation, and names prefixed with GL_ are reserved
+    * for use by Khronos.  Since every extension adds a name prefixed
+    * with GL_ (i.e., the name of the extension), that should be an
+    * error.  Names simply containing __ are dangerous to use, but should
+    * be allowed.
+    *
+    * A future version of the GLSL specification will clarify this.
+    */
+   if (strstr(identifier, "__")) {
+      glcpp_warning(loc, parser, "Macro names containing \"__\" are reserved "
+                    "for use by the implementation.\n");
+   }
+   if (strncmp(identifier, "GL_", 3) == 0) {
+      glcpp_error (loc, parser, "Macro names starting with \"GL_\" are reserved.\n");
+   }
+   if (strcmp(identifier, "defined") == 0) {
+      glcpp_error (loc, parser, "\"defined\" cannot be used as a macro name");
+   }
 }
 
 static int
-_macro_equal (macro_t *a, macro_t *b)
+_macro_equal(macro_t *a, macro_t *b)
 {
-	if (a->is_function != b->is_function)
-		return 0;
+   if (a->is_function != b->is_function)
+      return 0;
 
-	if (a->is_function) {
-		if (! _string_list_equal (a->parameters, b->parameters))
-			return 0;
-	}
+   if (a->is_function) {
+      if (! _string_list_equal (a->parameters, b->parameters))
+         return 0;
+   }
 
-	return _token_list_equal_ignoring_space (a->replacements,
-						 b->replacements);
+   return _token_list_equal_ignoring_space(a->replacements, b->replacements);
 }
 
 void
-_define_object_macro (glcpp_parser_t *parser,
-		      YYLTYPE *loc,
-		      const char *identifier,
-		      token_list_t *replacements)
+_define_object_macro(glcpp_parser_t *parser, YYLTYPE *loc,
+                     const char *identifier, token_list_t *replacements)
 {
-	macro_t *macro, *previous;
-
-	/* We define pre-defined macros before we've started parsing the
-         * actual file. So if there's no location defined yet, that's what
-         * were doing and we don't want to generate an error for using the
-         * reserved names. */
-	if (loc != NULL)
-		_check_for_reserved_macro_name(parser, loc, identifier);
-
-	macro = ralloc (parser, macro_t);
-
-	macro->is_function = 0;
-	macro->parameters = NULL;
-	macro->identifier = ralloc_strdup (macro, identifier);
-	macro->replacements = replacements;
-	ralloc_steal (macro, replacements);
-
-	previous = hash_table_find (parser->defines, identifier);
-	if (previous) {
-		if (_macro_equal (macro, previous)) {
-			ralloc_free (macro);
-			return;
-		}
-		glcpp_error (loc, parser, "Redefinition of macro %s\n",
-			     identifier);
-	}
-
-	hash_table_insert (parser->defines, macro, identifier);
+   macro_t *macro, *previous;
+
+   /* We define pre-defined macros before we've started parsing the actual
+    * file. So if there's no location defined yet, that's what were doing and
+    * we don't want to generate an error for using the reserved names. */
+   if (loc != NULL)
+      _check_for_reserved_macro_name(parser, loc, identifier);
+
+   macro = ralloc (parser, macro_t);
+
+   macro->is_function = 0;
+   macro->parameters = NULL;
+   macro->identifier = ralloc_strdup (macro, identifier);
+   macro->replacements = replacements;
+   ralloc_steal (macro, replacements);
+
+   previous = hash_table_find (parser->defines, identifier);
+   if (previous) {
+      if (_macro_equal (macro, previous)) {
+         ralloc_free (macro);
+         return;
+      }
+      glcpp_error (loc, parser, "Redefinition of macro %s\n",  identifier);
+   }
+
+   hash_table_insert (parser->defines, macro, identifier);
 }
 
 void
-_define_function_macro (glcpp_parser_t *parser,
-			YYLTYPE *loc,
-			const char *identifier,
-			string_list_t *parameters,
-			token_list_t *replacements)
+_define_function_macro(glcpp_parser_t *parser, YYLTYPE *loc,
+                       const char *identifier, string_list_t *parameters,
+                       token_list_t *replacements)
 {
-	macro_t *macro, *previous;
-	const char *dup;
+   macro_t *macro, *previous;
+   const char *dup;
 
-	_check_for_reserved_macro_name(parser, loc, identifier);
+   _check_for_reserved_macro_name(parser, loc, identifier);
 
         /* Check for any duplicate parameter names. */
-	if ((dup = _string_list_has_duplicate (parameters)) != NULL) {
-		glcpp_error (loc, parser, "Duplicate macro parameter \"%s\"",
-			     dup);
-	}
-
-	macro = ralloc (parser, macro_t);
-	ralloc_steal (macro, parameters);
-	ralloc_steal (macro, replacements);
-
-	macro->is_function = 1;
-	macro->parameters = parameters;
-	macro->identifier = ralloc_strdup (macro, identifier);
-	macro->replacements = replacements;
-	previous = hash_table_find (parser->defines, identifier);
-	if (previous) {
-		if (_macro_equal (macro, previous)) {
-			ralloc_free (macro);
-			return;
-		}
-		glcpp_error (loc, parser, "Redefinition of macro %s\n",
-			     identifier);
-	}
-
-	hash_table_insert (parser->defines, macro, identifier);
+   if ((dup = _string_list_has_duplicate (parameters)) != NULL) {
+      glcpp_error (loc, parser, "Duplicate macro parameter \"%s\"", dup);
+   }
+
+   macro = ralloc (parser, macro_t);
+   ralloc_steal (macro, parameters);
+   ralloc_steal (macro, replacements);
+
+   macro->is_function = 1;
+   macro->parameters = parameters;
+   macro->identifier = ralloc_strdup (macro, identifier);
+   macro->replacements = replacements;
+   previous = hash_table_find (parser->defines, identifier);
+   if (previous) {
+      if (_macro_equal (macro, previous)) {
+         ralloc_free (macro);
+         return;
+      }
+      glcpp_error (loc, parser, "Redefinition of macro %s\n", identifier);
+   }
+
+   hash_table_insert(parser->defines, macro, identifier);
 }
 
 static int
-glcpp_parser_lex (YYSTYPE *yylval, YYLTYPE *yylloc, glcpp_parser_t *parser)
+glcpp_parser_lex(YYSTYPE *yylval, YYLTYPE *yylloc, glcpp_parser_t *parser)
 {
-	token_node_t *node;
-	int ret;
-
-	if (parser->lex_from_list == NULL) {
-		ret = glcpp_lex (yylval, yylloc, parser->scanner);
-
-		/* XXX: This ugly block of code exists for the sole
-		 * purpose of converting a NEWLINE token into a SPACE
-		 * token, but only in the case where we have seen a
-		 * function-like macro name, but have not yet seen its
-		 * closing parenthesis.
-		 *
-		 * There's perhaps a more compact way to do this with
-		 * mid-rule actions in the grammar.
-		 *
-		 * I'm definitely not pleased with the complexity of
-		 * this code here.
-		 */
-		if (parser->newline_as_space)
-		{
-			if (ret == '(') {
-				parser->paren_count++;
-			} else if (ret == ')') {
-				parser->paren_count--;
-				if (parser->paren_count == 0)
-					parser->newline_as_space = 0;
-			} else if (ret == NEWLINE) {
-				ret = SPACE;
-			} else if (ret != SPACE) {
-				if (parser->paren_count == 0)
-					parser->newline_as_space = 0;
-			}
-		}
-		else if (parser->in_control_line)
-		{
-			if (ret == NEWLINE)
-				parser->in_control_line = 0;
-		}
-		else if (ret == DEFINE_TOKEN ||
-			 ret == UNDEF || ret == IF ||
-			 ret == IFDEF || ret == IFNDEF ||
-			 ret == ELIF || ret == ELSE ||
-			 ret == ENDIF || ret == HASH_TOKEN)
-		{
-			parser->in_control_line = 1;
-		}
-		else if (ret == IDENTIFIER)
-		{
-			macro_t *macro;
-			macro = hash_table_find (parser->defines,
-						 yylval->str);
-			if (macro && macro->is_function) {
-				parser->newline_as_space = 1;
-				parser->paren_count = 0;
-			}
-		}
-
-		return ret;
-	}
-
-	node = parser->lex_from_node;
-
-	if (node == NULL) {
-		ralloc_free (parser->lex_from_list);
-		parser->lex_from_list = NULL;
-		return NEWLINE;
-	}
-
-	*yylval = node->token->value;
-	ret = node->token->type;
-
-	parser->lex_from_node = node->next;
-
-	return ret;
+   token_node_t *node;
+   int ret;
+
+   if (parser->lex_from_list == NULL) {
+      ret = glcpp_lex(yylval, yylloc, parser->scanner);
+
+      /* XXX: This ugly block of code exists for the sole
+       * purpose of converting a NEWLINE token into a SPACE
+       * token, but only in the case where we have seen a
+       * function-like macro name, but have not yet seen its
+       * closing parenthesis.
+       *
+       * There's perhaps a more compact way to do this with
+       * mid-rule actions in the grammar.
+       *
+       * I'm definitely not pleased with the complexity of
+       * this code here.
+       */
+      if (parser->newline_as_space) {
+         if (ret == '(') {
+            parser->paren_count++;
+         } else if (ret == ')') {
+            parser->paren_count--;
+            if (parser->paren_count == 0)
+               parser->newline_as_space = 0;
+         } else if (ret == NEWLINE) {
+            ret = SPACE;
+         } else if (ret != SPACE) {
+            if (parser->paren_count == 0)
+               parser->newline_as_space = 0;
+         }
+      } else if (parser->in_control_line) {
+         if (ret == NEWLINE)
+            parser->in_control_line = 0;
+      }
+      else if (ret == DEFINE_TOKEN || ret == UNDEF || ret == IF ||
+               ret == IFDEF || ret == IFNDEF || ret == ELIF || ret == ELSE ||
+               ret == ENDIF || ret == HASH_TOKEN) {
+         parser->in_control_line = 1;
+      } else if (ret == IDENTIFIER) {
+         macro_t *macro;
+         macro = hash_table_find (parser->defines,
+                   yylval->str);
+         if (macro && macro->is_function) {
+            parser->newline_as_space = 1;
+            parser->paren_count = 0;
+         }
+      }
+
+      return ret;
+   }
+
+   node = parser->lex_from_node;
+
+   if (node == NULL) {
+      ralloc_free (parser->lex_from_list);
+      parser->lex_from_list = NULL;
+      return NEWLINE;
+   }
+
+   *yylval = node->token->value;
+   ret = node->token->type;
+
+   parser->lex_from_node = node->next;
+
+   return ret;
 }
 
 static void
-glcpp_parser_lex_from (glcpp_parser_t *parser, token_list_t *list)
+glcpp_parser_lex_from(glcpp_parser_t *parser, token_list_t *list)
 {
-	token_node_t *node;
+   token_node_t *node;
 
-	assert (parser->lex_from_list == NULL);
+   assert (parser->lex_from_list == NULL);
 
-	/* Copy list, eliminating any space tokens. */
-	parser->lex_from_list = _token_list_create (parser);
+   /* Copy list, eliminating any space tokens. */
+   parser->lex_from_list = _token_list_create (parser);
 
-	for (node = list->head; node; node = node->next) {
-		if (node->token->type == SPACE)
-			continue;
-		_token_list_append (parser->lex_from_list, node->token);
-	}
+   for (node = list->head; node; node = node->next) {
+      if (node->token->type == SPACE)
+         continue;
+      _token_list_append (parser->lex_from_list, node->token);
+   }
 
-	ralloc_free (list);
+   ralloc_free (list);
 
-	parser->lex_from_node = parser->lex_from_list->head;
+   parser->lex_from_node = parser->lex_from_list->head;
 
-	/* It's possible the list consisted of nothing but whitespace. */
-	if (parser->lex_from_node == NULL) {
-		ralloc_free (parser->lex_from_list);
-		parser->lex_from_list = NULL;
-	}
+   /* It's possible the list consisted of nothing but whitespace. */
+   if (parser->lex_from_node == NULL) {
+      ralloc_free (parser->lex_from_list);
+      parser->lex_from_list = NULL;
+   }
 }
 
 static void
-_glcpp_parser_skip_stack_push_if (glcpp_parser_t *parser, YYLTYPE *loc,
-				  int condition)
+_glcpp_parser_skip_stack_push_if(glcpp_parser_t *parser, YYLTYPE *loc,
+                                 int condition)
 {
-	skip_type_t current = SKIP_NO_SKIP;
-	skip_node_t *node;
-
-	if (parser->skip_stack)
-		current = parser->skip_stack->type;
-
-	node = ralloc (parser, skip_node_t);
-	node->loc = *loc;
-
-	if (current == SKIP_NO_SKIP) {
-		if (condition)
-			node->type = SKIP_NO_SKIP;
-		else
-			node->type = SKIP_TO_ELSE;
-	} else {
-		node->type = SKIP_TO_ENDIF;
-	}
-
-	node->has_else = false;
-	node->next = parser->skip_stack;
-	parser->skip_stack = node;
+   skip_type_t current = SKIP_NO_SKIP;
+   skip_node_t *node;
+
+   if (parser->skip_stack)
+      current = parser->skip_stack->type;
+
+   node = ralloc (parser, skip_node_t);
+   node->loc = *loc;
+
+   if (current == SKIP_NO_SKIP) {
+      if (condition)
+         node->type = SKIP_NO_SKIP;
+      else
+         node->type = SKIP_TO_ELSE;
+   } else {
+      node->type = SKIP_TO_ENDIF;
+   }
+
+   node->has_else = false;
+   node->next = parser->skip_stack;
+   parser->skip_stack = node;
 }
 
 static void
-_glcpp_parser_skip_stack_change_if (glcpp_parser_t *parser, YYLTYPE *loc,
-				    const char *type, int condition)
+_glcpp_parser_skip_stack_change_if(glcpp_parser_t *parser, YYLTYPE *loc,
+                                   const char *type, int condition)
 {
-	if (parser->skip_stack == NULL) {
-		glcpp_error (loc, parser, "#%s without #if\n", type);
-		return;
-	}
-
-	if (parser->skip_stack->type == SKIP_TO_ELSE) {
-		if (condition)
-			parser->skip_stack->type = SKIP_NO_SKIP;
-	} else {
-		parser->skip_stack->type = SKIP_TO_ENDIF;
-	}
+   if (parser->skip_stack == NULL) {
+      glcpp_error (loc, parser, "#%s without #if\n", type);
+      return;
+   }
+
+   if (parser->skip_stack->type == SKIP_TO_ELSE) {
+      if (condition)
+         parser->skip_stack->type = SKIP_NO_SKIP;
+   } else {
+      parser->skip_stack->type = SKIP_TO_ENDIF;
+   }
 }
 
 static void
-_glcpp_parser_skip_stack_pop (glcpp_parser_t *parser, YYLTYPE *loc)
+_glcpp_parser_skip_stack_pop(glcpp_parser_t *parser, YYLTYPE *loc)
 {
-	skip_node_t *node;
+   skip_node_t *node;
 
-	if (parser->skip_stack == NULL) {
-		glcpp_error (loc, parser, "#endif without #if\n");
-		return;
-	}
+   if (parser->skip_stack == NULL) {
+      glcpp_error (loc, parser, "#endif without #if\n");
+      return;
+   }
 
-	node = parser->skip_stack;
-	parser->skip_stack = node->next;
-	ralloc_free (node);
+   node = parser->skip_stack;
+   parser->skip_stack = node->next;
+   ralloc_free (node);
 }
 
 static void
@@ -2349,206 +2279,205 @@ _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t versio
                                          const char *es_identifier,
                                          bool explicitly_set)
 {
-	const struct gl_extensions *extensions = parser->extensions;
-
-	if (parser->version_resolved)
-		return;
-
-	parser->version_resolved = true;
+   const struct gl_extensions *extensions = parser->extensions;
+
+   if (parser->version_resolved)
+      return;
+
+   parser->version_resolved = true;
+
+   add_builtin_define (parser, "__VERSION__", version);
+
+   parser->is_gles = (version == 100) ||
+                     (es_identifier && (strcmp(es_identifier, "es") == 0));
+
+   /* Add pre-defined macros. */
+   if (parser->is_gles) {
+      add_builtin_define(parser, "GL_ES", 1);
+      add_builtin_define(parser, "GL_EXT_separate_shader_objects", 1);
+      add_builtin_define(parser, "GL_EXT_draw_buffers", 1);
 
-	add_builtin_define (parser, "__VERSION__", version);
-
-	parser->is_gles = (version == 100) ||
-			   (es_identifier &&
-			    (strcmp(es_identifier, "es") == 0));
+      if (extensions != NULL) {
+         if (extensions->OES_EGL_image_external)
+            add_builtin_define(parser, "GL_OES_EGL_image_external", 1);
+         if (extensions->OES_standard_derivatives)
+            add_builtin_define(parser, "GL_OES_standard_derivatives", 1);
+         if (extensions->ARB_texture_multisample)
+            add_builtin_define(parser, "GL_OES_texture_storage_multisample_2d_array", 1);
+         if (extensions->ARB_blend_func_extended)
+            add_builtin_define(parser, "GL_EXT_blend_func_extended", 1);
 
-	/* Add pre-defined macros. */
-	if (parser->is_gles) {
-	   add_builtin_define(parser, "GL_ES", 1);
-           add_builtin_define(parser, "GL_EXT_separate_shader_objects", 1);
-           add_builtin_define(parser, "GL_EXT_draw_buffers", 1);
+         if (version >= 310) {
+            if (extensions->ARB_shader_image_load_store)
+               add_builtin_define(parser, "GL_OES_shader_image_atomic", 1);
 
-	   if (extensions != NULL) {
-	      if (extensions->OES_EGL_image_external)
-	         add_builtin_define(parser, "GL_OES_EGL_image_external", 1);
-              if (extensions->OES_standard_derivatives)
-                 add_builtin_define(parser, "GL_OES_standard_derivatives", 1);
-              if (extensions->ARB_texture_multisample)
-                 add_builtin_define(parser, "GL_OES_texture_storage_multisample_2d_array", 1);
-              if (extensions->ARB_blend_func_extended)
-                 add_builtin_define(parser, "GL_EXT_blend_func_extended", 1);
+            if (extensions->OES_geometry_shader) {
+               add_builtin_define(parser, "GL_OES_geometry_point_size", 1);
+               add_builtin_define(parser, "GL_OES_geometry_shader", 1);
+            }
+            if (extensions->ARB_gpu_shader5) {
+               add_builtin_define(parser, "GL_EXT_gpu_shader5", 1);
+               add_builtin_define(parser, "GL_OES_gpu_shader5", 1);
+            }
+            if (extensions->OES_texture_buffer) {
+               add_builtin_define(parser, "GL_EXT_texture_buffer", 1);
+               add_builtin_define(parser, "GL_OES_texture_buffer", 1);
+            }
+         }
+      }
+   } else {
+      add_builtin_define(parser, "GL_ARB_draw_buffers", 1);
+      add_builtin_define(parser, "GL_ARB_enhanced_layouts", 1);
+      add_builtin_define(parser, "GL_ARB_separate_shader_objects", 1);
+      add_builtin_define(parser, "GL_ARB_texture_rectangle", 1);
+      add_builtin_define(parser, "GL_AMD_shader_trinary_minmax", 1);
 
-              if (version >= 310) {
-                 if (extensions->ARB_shader_image_load_store)
-                    add_builtin_define(parser, "GL_OES_shader_image_atomic", 1);
+      if (extensions != NULL) {
+         if (extensions->EXT_texture_array)
+            add_builtin_define(parser, "GL_EXT_texture_array", 1);
 
-                 if (extensions->OES_geometry_shader) {
-                    add_builtin_define(parser, "GL_OES_geometry_point_size", 1);
-                    add_builtin_define(parser, "GL_OES_geometry_shader", 1);
-                 }
-                 if (extensions->ARB_gpu_shader5) {
-                    add_builtin_define(parser, "GL_EXT_gpu_shader5", 1);
-                    add_builtin_define(parser, "GL_OES_gpu_shader5", 1);
-                 }
-                 if (extensions->OES_texture_buffer) {
-                    add_builtin_define(parser, "GL_EXT_texture_buffer", 1);
-                    add_builtin_define(parser, "GL_OES_texture_buffer", 1);
-                 }
-              }
-	   }
-	} else {
-	   add_builtin_define(parser, "GL_ARB_draw_buffers", 1);
-           add_builtin_define(parser, "GL_ARB_enhanced_layouts", 1);
-           add_builtin_define(parser, "GL_ARB_separate_shader_objects", 1);
-	   add_builtin_define(parser, "GL_ARB_texture_rectangle", 1);
-           add_builtin_define(parser, "GL_AMD_shader_trinary_minmax", 1);
+         if (extensions->ARB_arrays_of_arrays)
+             add_builtin_define(parser, "GL_ARB_arrays_of_arrays", 1);
 
+         if (extensions->ARB_fragment_coord_conventions) {
+            add_builtin_define(parser, "GL_ARB_fragment_coord_conventions",
+                               1);
+         }
 
-	   if (extensions != NULL) {
-	      if (extensions->EXT_texture_array)
-	         add_builtin_define(parser, "GL_EXT_texture_array", 1);
+         if (extensions->ARB_fragment_layer_viewport)
+            add_builtin_define(parser, "GL_ARB_fragment_layer_viewport", 1);
 
-	      if (extensions->ARB_arrays_of_arrays)
-	          add_builtin_define(parser, "GL_ARB_arrays_of_arrays", 1);
+         if (extensions->ARB_explicit_attrib_location)
+            add_builtin_define(parser, "GL_ARB_explicit_attrib_location", 1);
 
-	      if (extensions->ARB_fragment_coord_conventions)
-	         add_builtin_define(parser, "GL_ARB_fragment_coord_conventions",
-				    1);
+         if (extensions->ARB_explicit_uniform_location)
+            add_builtin_define(parser, "GL_ARB_explicit_uniform_location", 1);
 
-              if (extensions->ARB_fragment_layer_viewport)
-                 add_builtin_define(parser, "GL_ARB_fragment_layer_viewport", 1);
+         if (extensions->ARB_shader_texture_lod)
+            add_builtin_define(parser, "GL_ARB_shader_texture_lod", 1);
 
-	      if (extensions->ARB_explicit_attrib_location)
-	         add_builtin_define(parser, "GL_ARB_explicit_attrib_location", 1);
+         if (extensions->ARB_draw_instanced)
+            add_builtin_define(parser, "GL_ARB_draw_instanced", 1);
 
-	      if (extensions->ARB_explicit_uniform_location)
-	         add_builtin_define(parser, "GL_ARB_explicit_uniform_location", 1);
+         if (extensions->ARB_conservative_depth) {
+            add_builtin_define(parser, "GL_AMD_conservative_depth", 1);
+            add_builtin_define(parser, "GL_ARB_conservative_depth", 1);
+         }
 
-	      if (extensions->ARB_shader_texture_lod)
-	         add_builtin_define(parser, "GL_ARB_shader_texture_lod", 1);
+         if (extensions->ARB_shader_bit_encoding)
+            add_builtin_define(parser, "GL_ARB_shader_bit_encoding", 1);
 
-	      if (extensions->ARB_draw_instanced)
-	         add_builtin_define(parser, "GL_ARB_draw_instanced", 1);
+         if (extensions->ARB_shader_clock)
+            add_builtin_define(parser, "GL_ARB_shader_clock", 1);
 
-	      if (extensions->ARB_conservative_depth) {
-	         add_builtin_define(parser, "GL_AMD_conservative_depth", 1);
-	         add_builtin_define(parser, "GL_ARB_conservative_depth", 1);
-	      }
+         if (extensions->ARB_uniform_buffer_object)
+            add_builtin_define(parser, "GL_ARB_uniform_buffer_object", 1);
 
-	      if (extensions->ARB_shader_bit_encoding)
-	         add_builtin_define(parser, "GL_ARB_shader_bit_encoding", 1);
+         if (extensions->ARB_texture_cube_map_array)
+            add_builtin_define(parser, "GL_ARB_texture_cube_map_array", 1);
 
-	      if (extensions->ARB_shader_clock)
-	         add_builtin_define(parser, "GL_ARB_shader_clock", 1);
+         if (extensions->ARB_shading_language_packing)
+            add_builtin_define(parser, "GL_ARB_shading_language_packing", 1);
 
-	      if (extensions->ARB_uniform_buffer_object)
-	         add_builtin_define(parser, "GL_ARB_uniform_buffer_object", 1);
+         if (extensions->ARB_texture_multisample)
+            add_builtin_define(parser, "GL_ARB_texture_multisample", 1);
 
-	      if (extensions->ARB_texture_cube_map_array)
-	         add_builtin_define(parser, "GL_ARB_texture_cube_map_array", 1);
+         if (extensions->ARB_texture_query_levels)
+            add_builtin_define(parser, "GL_ARB_texture_query_levels", 1);
 
-	      if (extensions->ARB_shading_language_packing)
-	         add_builtin_define(parser, "GL_ARB_shading_language_packing", 1);
+         if (extensions->ARB_texture_query_lod)
+            add_builtin_define(parser, "GL_ARB_texture_query_lod", 1);
 
-	      if (extensions->ARB_texture_multisample)
-	         add_builtin_define(parser, "GL_ARB_texture_multisample", 1);
+         if (extensions->ARB_gpu_shader5)
+            add_builtin_define(parser, "GL_ARB_gpu_shader5", 1);
 
-	      if (extensions->ARB_texture_query_levels)
-	         add_builtin_define(parser, "GL_ARB_texture_query_levels", 1);
+         if (extensions->ARB_gpu_shader_fp64)
+            add_builtin_define(parser, "GL_ARB_gpu_shader_fp64", 1);
 
-	      if (extensions->ARB_texture_query_lod)
-	         add_builtin_define(parser, "GL_ARB_texture_query_lod", 1);
+         if (extensions->ARB_vertex_attrib_64bit)
+            add_builtin_define(parser, "GL_ARB_vertex_attrib_64bit", 1);
 
-	      if (extensions->ARB_gpu_shader5)
-	         add_builtin_define(parser, "GL_ARB_gpu_shader5", 1);
+         if (extensions->AMD_vertex_shader_layer)
+            add_builtin_define(parser, "GL_AMD_vertex_shader_layer", 1);
 
-              if (extensions->ARB_gpu_shader_fp64)
-                 add_builtin_define(parser, "GL_ARB_gpu_shader_fp64", 1);
+         if (extensions->AMD_vertex_shader_viewport_index)
+            add_builtin_define(parser, "GL_AMD_vertex_shader_viewport_index", 1);
 
-	      if (extensions->ARB_vertex_attrib_64bit)
-	         add_builtin_define(parser, "GL_ARB_vertex_attrib_64bit", 1);
+         if (extensions->ARB_shading_language_420pack)
+            add_builtin_define(parser, "GL_ARB_shading_language_420pack", 1);
 
-	      if (extensions->AMD_vertex_shader_layer)
-	         add_builtin_define(parser, "GL_AMD_vertex_shader_layer", 1);
+         if (extensions->ARB_sample_shading)
+            add_builtin_define(parser, "GL_ARB_sample_shading", 1);
 
-	      if (extensions->AMD_vertex_shader_viewport_index)
-	         add_builtin_define(parser, "GL_AMD_vertex_shader_viewport_index", 1);
+         if (extensions->ARB_texture_gather)
+            add_builtin_define(parser, "GL_ARB_texture_gather", 1);
 
-	      if (extensions->ARB_shading_language_420pack)
-	         add_builtin_define(parser, "GL_ARB_shading_language_420pack", 1);
+         if (extensions->ARB_shader_atomic_counters)
+            add_builtin_define(parser, "GL_ARB_shader_atomic_counters", 1);
 
-	      if (extensions->ARB_sample_shading)
-	         add_builtin_define(parser, "GL_ARB_sample_shading", 1);
+         if (extensions->ARB_shader_atomic_counter_ops)
+            add_builtin_define(parser, "GL_ARB_shader_atomic_counter_ops", 1);
 
-	      if (extensions->ARB_texture_gather)
-	         add_builtin_define(parser, "GL_ARB_texture_gather", 1);
+         if (extensions->ARB_viewport_array)
+            add_builtin_define(parser, "GL_ARB_viewport_array", 1);
 
-	      if (extensions->ARB_shader_atomic_counters)
-	         add_builtin_define(parser, "GL_ARB_shader_atomic_counters", 1);
+         if (extensions->ARB_compute_shader)
+            add_builtin_define(parser, "GL_ARB_compute_shader", 1);
 
-	      if (extensions->ARB_shader_atomic_counter_ops)
-	         add_builtin_define(parser, "GL_ARB_shader_atomic_counter_ops", 1);
+         if (extensions->ARB_shader_image_load_store)
+            add_builtin_define(parser, "GL_ARB_shader_image_load_store", 1);
 
-	      if (extensions->ARB_viewport_array)
-	         add_builtin_define(parser, "GL_ARB_viewport_array", 1);
+         if (extensions->ARB_shader_image_size)
+            add_builtin_define(parser, "GL_ARB_shader_image_size", 1);
 
-              if (extensions->ARB_compute_shader)
-                 add_builtin_define(parser, "GL_ARB_compute_shader", 1);
+         if (extensions->ARB_shader_texture_image_samples)
+            add_builtin_define(parser, "GL_ARB_shader_texture_image_samples", 1);
 
-	      if (extensions->ARB_shader_image_load_store)
-	         add_builtin_define(parser, "GL_ARB_shader_image_load_store", 1);
+         if (extensions->ARB_derivative_control)
+            add_builtin_define(parser, "GL_ARB_derivative_control", 1);
 
-              if (extensions->ARB_shader_image_size)
-                 add_builtin_define(parser, "GL_ARB_shader_image_size", 1);
+         if (extensions->ARB_shader_precision)
+            add_builtin_define(parser, "GL_ARB_shader_precision", 1);
 
-              if (extensions->ARB_shader_texture_image_samples)
-                 add_builtin_define(parser, "GL_ARB_shader_texture_image_samples", 1);
+         if (extensions->ARB_shader_storage_buffer_object)
+            add_builtin_define(parser, "GL_ARB_shader_storage_buffer_object", 1);
 
-              if (extensions->ARB_derivative_control)
-                 add_builtin_define(parser, "GL_ARB_derivative_control", 1);
+         if (extensions->ARB_tessellation_shader)
+            add_builtin_define(parser, "GL_ARB_tessellation_shader", 1);
 
-              if (extensions->ARB_shader_precision)
-                 add_builtin_define(parser, "GL_ARB_shader_precision", 1);
+         if (extensions->ARB_shader_subroutine)
+            add_builtin_define(parser, "GL_ARB_shader_subroutine", 1);
 
-	      if (extensions->ARB_shader_storage_buffer_object)
-	         add_builtin_define(parser, "GL_ARB_shader_storage_buffer_object", 1);
+         if (extensions->ARB_shader_draw_parameters)
+            add_builtin_define(parser, "GL_ARB_shader_draw_parameters", 1);
+      }
+   }
 
-	      if (extensions->ARB_tessellation_shader)
-	         add_builtin_define(parser, "GL_ARB_tessellation_shader", 1);
+   if (extensions != NULL) {
+      if (extensions->EXT_shader_integer_mix)
+         add_builtin_define(parser, "GL_EXT_shader_integer_mix", 1);
 
-              if (extensions->ARB_shader_subroutine)
-                 add_builtin_define(parser, "GL_ARB_shader_subroutine", 1);
+      if (extensions->EXT_shader_samples_identical)
+         add_builtin_define(parser, "GL_EXT_shader_samples_identical", 1);
+   }
 
-              if (extensions->ARB_shader_draw_parameters)
-                 add_builtin_define(parser, "GL_ARB_shader_draw_parameters", 1);
-	   }
-	}
+   if (version >= 150)
+      add_builtin_define(parser, "GL_core_profile", 1);
 
-	if (extensions != NULL) {
-	   if (extensions->EXT_shader_integer_mix)
-	      add_builtin_define(parser, "GL_EXT_shader_integer_mix", 1);
+   /* Currently, all ES2/ES3 implementations support highp in the
+    * fragment shader, so we always define this macro in ES2/ES3.
+    * If we ever get a driver that doesn't support highp, we'll
+    * need to add a flag to the gl_context and check that here.
+    */
+   if (version >= 130 || parser->is_gles)
+      add_builtin_define (parser, "GL_FRAGMENT_PRECISION_HIGH", 1);
 
-	   if (extensions->EXT_shader_samples_identical)
-	      add_builtin_define(parser, "GL_EXT_shader_samples_identical", 1);
-	}
-
-	if (version >= 150)
-		add_builtin_define(parser, "GL_core_profile", 1);
-
-	/* Currently, all ES2/ES3 implementations support highp in the
-	 * fragment shader, so we always define this macro in ES2/ES3.
-	 * If we ever get a driver that doesn't support highp, we'll
-	 * need to add a flag to the gl_context and check that here.
-	 */
-	if (version >= 130 || parser->is_gles)
-		add_builtin_define (parser, "GL_FRAGMENT_PRECISION_HIGH", 1);
-
-	if (explicitly_set) {
-	   ralloc_asprintf_rewrite_tail (&parser->output, &parser->output_length,
-					 "#version %" PRIiMAX "%s%s", version,
-					 es_identifier ? " " : "",
-					 es_identifier ? es_identifier : "");
-	}
+   if (explicitly_set) {
+      ralloc_asprintf_rewrite_tail(&parser->output, &parser->output_length,
+                                   "#version %" PRIiMAX "%s%s", version,
+                                   es_identifier ? " " : "",
+                                   es_identifier ? es_identifier : "");
+   }
 }
 
 /* GLSL version if no version is explicitly specified. */
@@ -2560,10 +2489,9 @@ _glcpp_parser_handle_version_declaration(glcpp_parser_t *parser, intmax_t versio
 void
 glcpp_parser_resolve_implicit_version(glcpp_parser_t *parser)
 {
-	int language_version = parser->api == API_OPENGLES2 ?
-			       IMPLICIT_GLSL_ES_VERSION :
-			       IMPLICIT_GLSL_VERSION;
+   int language_version = parser->api == API_OPENGLES2 ?
+                          IMPLICIT_GLSL_ES_VERSION : IMPLICIT_GLSL_VERSION;
 
-	_glcpp_parser_handle_version_declaration(parser, language_version,
-						 NULL, false);
+   _glcpp_parser_handle_version_declaration(parser, language_version,
+                                            NULL, false);
 }
-- 
2.5.5



More information about the mesa-dev mailing list