diff options
| author | Carl Worth <cworth@cworth.org> | 2010-08-11 13:59:22 -0700 |
|---|---|---|
| committer | Carl Worth <cworth@cworth.org> | 2010-08-11 14:38:04 -0700 |
| commit | 253cad3f424f71f6984431e5edbde1694ccfae3f (patch) | |
| tree | b9930838da3da94b060c6589448ab48e129a3c42 /src/glsl/glcpp/glcpp-parse.y | |
| parent | 624dd585c72103e5bffbc600cdf7bdfba5305a15 (diff) | |
glcpp: Add an explicit diagnostic for #if with no expression.
This is more clear than the previously-generated diagnostic which was
something confusing like "enexpected newline".
This change makse test 080-if-witout-expression.c now pass.
Diffstat (limited to 'src/glsl/glcpp/glcpp-parse.y')
| -rw-r--r-- | src/glsl/glcpp/glcpp-parse.y | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y index 795030ecfe..0e0d9d412b 100644 --- a/src/glsl/glcpp/glcpp-parse.y +++ b/src/glsl/glcpp/glcpp-parse.y @@ -241,6 +241,16 @@ control_line: parser->skip_stack->type = SKIP_TO_ENDIF; } } +| HASH_IF NEWLINE { + /* #if without an expression is only an error if we + * are not skipping */ + if (parser->skip_stack == NULL || + parser->skip_stack->type == SKIP_NO_SKIP) + { + glcpp_error(& @1, parser, "#if with no expression"); + } + _glcpp_parser_skip_stack_push_if (parser, & @1, 0); + } | HASH_IFDEF IDENTIFIER junk NEWLINE { macro_t *macro = hash_table_find (parser->defines, $2); talloc_free ($2); |