This patch is Reviewed-by: Ian Romanick <ian.d.roman...@intel.com>
On 03/20/2015 06:59 AM, Jose Fonseca wrote: > This addresses > > ...\glsl_parser.cpp(...) : warning C4065: switch statement contains > 'default' but no 'case' labels > > This is on code generated by bison, which we have little control. > > It seems useful to have this warning otherwise enabled. > --- > src/glsl/glsl_parser.yy | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy > index 90c216e..aceb3b9 100644 > --- a/src/glsl/glsl_parser.yy > +++ b/src/glsl/glsl_parser.yy > @@ -34,6 +34,10 @@ > #include "glsl_types.h" > #include "main/context.h" > > +#ifdef _MSC_VER > +#pragma warning( disable : 4065 ) // switch statement contains 'default' but > no 'case' labels > +#endif > + > #undef yyerror > > static void yyerror(YYLTYPE *loc, _mesa_glsl_parse_state *st, const char > *msg) > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
