[Mesa-dev] [PATCH 02/15] glsl: Disable MSVC switch warning on a per-file basis.
Ian Romanick
idr at freedesktop.org
Fri Mar 20 14:20:58 PDT 2015
This patch is
Reviewed-by: Ian Romanick <ian.d.romanick at 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)
>
More information about the mesa-dev
mailing list