glcpp: Add a more descriptive comment for the SKIP state manipulation
authorCarl Worth <cworth@cworth.org>
Thu, 19 Dec 2013 23:14:19 +0000 (15:14 -0800)
committerCarl Worth <cworth@cworth.org>
Thu, 2 Jan 2014 22:15:24 +0000 (14:15 -0800)
commit61cea49014df6dcf3aa41ad18b2e998e6921320f
tree777f98c774da40496bc4d03e024631e20dab66f2
parent5a51c1b01a16d3256f9769a76d8293fea5853b1f
glcpp: Add a more descriptive comment for the SKIP state manipulation

Two things make this code confusing:

1. The uncharacteristic manipulation of lexer start state outside of
   flex rules.

2. The confusing semantics of the skip_stack (including the
   "lexing_if" override and the SKIP_NO_SKIP state).

This new comment is intended to bring a bit more clarity for any readers.

There is no intended beahvioral change to the code here. The actual code
changes include better indentation to avoid an excessively-long line, and
using the more descriptive INITIAL rather than 0.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/glsl/glcpp/glcpp-lex.l