[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