On 12/17/2013 07:37 AM, Erik Faye-Lund wrote: > The preprocessor currently accepts multiple else/elif-groups > per if-section. The GLSL-preprocessor is defined by the C++ > specification, which defines the following parse-rule: > > if-section: > if-group elif-groups(opt) else-group(opt) endif-line > > This clearly only allows a single else-group, that has to come > after any elif-groups. > > So let's modify the code to follow the specification. Add test > to prevent regressions. > --- > > Here's a resend of an older patch (original > <[email protected]>), this time > with a better commit message, and a similar treatment for elif-after-else.
This looks good to me. Thanks! Reviewed-by: Kenneth Graunke <[email protected]> Presumably Ian or Carl will commit it today... _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
