Re: [Mesa-dev] [PATCH 2/4] glsl: Switch from the deprecated YYLEX_PARAM to %lex-param.

2013-07-29 Thread Matt Turner
On Mon, Jul 29, 2013 at 8:44 PM, Nikita Malyavin wrote: > Kenneth, maybe it's better to mark _mesa_glsl_lex as inline? The compiler will figure it out. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinf

Re: [Mesa-dev] [PATCH 2/4] glsl: Switch from the deprecated YYLEX_PARAM to %lex-param.

2013-07-29 Thread Nikita Malyavin
Kenneth, maybe it's better to mark _mesa_glsl_lex as inline? 2013/7/30 Kenneth Graunke > YYLEX_PARAM is no longer supported as of Bison 3.0. Instead, the Bison > developers recommend using %lex-param. > > %lex-param takes a type and variable name, similar to %parse-param, > so you can't pass a

[Mesa-dev] [PATCH 2/4] glsl: Switch from the deprecated YYLEX_PARAM to %lex-param.

2013-07-29 Thread Kenneth Graunke
YYLEX_PARAM is no longer supported as of Bison 3.0. Instead, the Bison developers recommend using %lex-param. %lex-param takes a type and variable name, similar to %parse-param, so you can't pass an arbitrary expression like state->scanner. But Flex insists on passing the actual scanner object,