diff options
| author | Timothy Arceri <[email protected]> | 2016-10-21 16:50:52 +1100 |
|---|---|---|
| committer | Timothy Arceri <[email protected]> | 2016-10-24 21:40:39 +1100 |
| commit | 6dbe8a1b9fd750b4c1bb600a0bb43129d95e6eca (patch) | |
| tree | e94d879f69543871534f59c3ffbfb6adf49efdce /src/mesa/program/program_lexer.l | |
| parent | 907ace57986733add2aebfa9dd7c83c67efed70e (diff) | |
glsl/mesa: remove unused namespace support from the symbol table
Namespace support seems to have been unused for a very long time.
Previously the hash table entry was never removed and the symbol name
wasn't freed until the symbol table was destroyed.
In theory this could reduced the number of times we need to copy a string
as duplicate names are reused. However in practice there is likely only a
limited number of symbols that are the same and this is likely to cause
other less than optimal behaviour such as the hash_table continuously
growing.
Along with dropping namespace support this change removes entries from
the hash table as they become unused.
Reviewed-by: Samuel Iglesias Gonsálvez <[email protected]>
Diffstat (limited to 'src/mesa/program/program_lexer.l')
| -rw-r--r-- | src/mesa/program/program_lexer.l | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/program/program_lexer.l b/src/mesa/program/program_lexer.l index bb169b930ce..dee66cbf30a 100644 --- a/src/mesa/program/program_lexer.l +++ b/src/mesa/program/program_lexer.l @@ -123,7 +123,7 @@ handle_ident(struct asm_parser_state *state, const char *text, YYSTYPE *lval) { lval->string = strdup(text); - return (_mesa_symbol_table_find_symbol(state->st, 0, text) == NULL) + return (_mesa_symbol_table_find_symbol(state->st, text) == NULL) ? IDENTIFIER : USED_IDENTIFIER; } |