diff options
author | Kenneth Graunke <kenneth@whitecape.org> | 2011-01-15 15:20:11 -0800 |
---|---|---|
committer | Kenneth Graunke <kenneth@whitecape.org> | 2011-01-21 15:41:19 -0800 |
commit | 6ecee54a9aecc120cb68b02f7e14dcac86b9eca2 (patch) | |
tree | 439cfc92256b2463d775e9570290b558a2316d4d | |
parent | e256e4743c3f8f924f0d191759d9428f33f3e329 (diff) | |
download | external_mesa3d-6ecee54a9aecc120cb68b02f7e14dcac86b9eca2.zip external_mesa3d-6ecee54a9aecc120cb68b02f7e14dcac86b9eca2.tar.gz external_mesa3d-6ecee54a9aecc120cb68b02f7e14dcac86b9eca2.tar.bz2 |
glcpp: Remove use of talloc reference counting.
We almost always want to simply steal; we only need to copy when copying
a token list (in which case we're already cloning stuff anyway).
-rw-r--r-- | src/glsl/glcpp/glcpp-parse.y | 20 |
1 files changed, 7 insertions, 13 deletions
diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y index e71c0e8..dacd321 100644 --- a/src/glsl/glcpp/glcpp-parse.y +++ b/src/glsl/glcpp/glcpp-parse.y @@ -89,10 +89,7 @@ _token_create_ival (void *ctx, int type, int ival); static token_list_t * _token_list_create (void *ctx); -/* Note: This function adds a talloc_reference() to token. - * - * You may want to talloc_unlink any current reference if you no - * longer need it. */ +/* Note: This function calls talloc_steal on token. */ static void _token_list_append (token_list_t *list, token_t *token); @@ -479,12 +476,10 @@ conditional_tokens: conditional_token { $$ = _token_list_create (parser); _token_list_append ($$, $1); - talloc_unlink (parser, $1); } | conditional_tokens conditional_token { $$ = $1; _token_list_append ($$, $2); - talloc_unlink (parser, $2); } ; @@ -493,12 +488,10 @@ pp_tokens: parser->space_tokens = 1; $$ = _token_list_create (parser); _token_list_append ($$, $1); - talloc_unlink (parser, $1); } | pp_tokens preprocessing_token { $$ = $1; _token_list_append ($$, $2); - talloc_unlink (parser, $2); } ; @@ -764,7 +757,7 @@ _token_list_append (token_list_t *list, token_t *token) token_node_t *node; node = talloc (list, token_node_t); - node->token = talloc_reference (list, token); + node->token = talloc_steal (list, token); node->next = NULL; @@ -805,8 +798,11 @@ _token_list_copy (void *ctx, token_list_t *other) return NULL; copy = _token_list_create (ctx); - for (node = other->head; node; node = node->next) - _token_list_append (copy, node->token); + for (node = other->head; node; node = node->next) { + token_t *new_token = talloc (copy, token_t); + *new_token = *node->token; + _token_list_append (copy, new_token); + } return copy; } @@ -1081,8 +1077,6 @@ static void add_builtin_define(glcpp_parser_t *parser, list = _token_list_create(parser); _token_list_append(list, tok); _define_object_macro(parser, NULL, name, list); - - talloc_unlink(parser, tok); } glcpp_parser_t * |