glcpp: Avoid accidental token pasting in preprocessed result.
authorCarl Worth <cworth@cworth.org>
Tue, 20 Jul 2010 23:44:03 +0000 (16:44 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Wed, 21 Jul 2010 00:01:12 +0000 (17:01 -0700)
commite1acbfca322c4ac720707ec8d3fda08fab65a30b
tree2ff0280eab03abc0c9c8655f050ea2ecb9567bdd
parent942ccc517012e360a7e30d3322331c8450dda022
glcpp: Avoid accidental token pasting in preprocessed result.

Consider this test case:

#define EMPTY
int foo = 1+EMPTY+4;

The expression should compile as the sequence of tokens 1, PLUS,
UNARY_POSITIVE, 4. But glcpp has been failing for this case since it
results in the string "1++4" which a compiler correctly sees as a
syntax error, (1, POST_INCREMENT, 4).

We fix this by changing any macro with an empty definition to result
in a single SPACE token rather than nothing. This then gives "1+ +4"
which compiles correctly.

This commit does touch up the two existing test cases which already
have empty macros, (to add the space to the expected result).

It also adds a new test case to exercise the above scenario.
src/glsl/glcpp/glcpp-parse.y
src/glsl/glcpp/tests/008-define-empty.c.expected
src/glsl/glcpp/tests/011-define-func-empty.c.expected
src/glsl/glcpp/tests/068-accidental-pasting.c [new file with mode: 0644]
src/glsl/glcpp/tests/068-accidental-pasting.c.expected [new file with mode: 0644]