glsl/glcpp: Don't choke on an empty pragma
authorCarl Worth <cworth@cworth.org>
Fri, 13 Jun 2014 22:53:39 +0000 (15:53 -0700)
committerCarl Worth <cworth@cworth.org>
Wed, 9 Jul 2014 19:05:14 +0000 (12:05 -0700)
commit0e12cd795474fcc38bae7f92b870f71fe47ca0bb
treea185952e35166142f13cdfcdff456361ae04e6e9
parent43047384c331a9240339563704fe5d97ebe599d7
glsl/glcpp: Don't choke on an empty pragma

The lexer was insisting that there be at least one character after "#pragma"
and before the end of the line. This caused an error for a line consisting
only of "#pragma" which volates at least the following sentence from the GLSL
ES Specification 3.00.4:

The scope as well as the effect of the optimize and debug pragmas is
implementation-dependent except that their use must not generate an
error. [Page 12 (Page 28 of PDF)]

and likely the following sentence from that specification and also in
GLSLangSpec 4.30.6:

If an implementation does not recognize the tokens following #pragma,
then it will ignore that pragma.

Add a "make check" test to ensure no future regressions.

This change fixes at least part of the following Khronos GLES3 CTS test:

preprocessor.pragmas.pragma_vertex

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/glsl/glcpp/glcpp-lex.l
src/glsl/glcpp/tests/127-pragma-empty.c [new file with mode: 0644]
src/glsl/glcpp/tests/127-pragma-empty.c.expected [new file with mode: 0644]