[Mesa-dev] [PATCH] i965: Rename INTEL_DEBUG=vec4vs to INTEL_DEBUG=vec4.

[Jason Ekstrand]

Acked-by: Jason Ekstrand <jason.ekstrand at intel.com>

On Wed, Aug 26, 2015 at 2:43 AM, Kenneth Graunke <kenneth at whitecape.org> wrote:
> driParseDebugString() doesn't have actual code to parse comma separated
> lists (or any other supported options?); instead it dumbly uses strstr().
>
> This means that INTEL_DEBUG="vec4vs" will trigger both DEBUG_VEC4VS and
> DEBUG_VS, as "vs" is also a substring.
>
> We should probably improve the driconf parsing, but for now, just rename
> the option so it's usable in the meantime.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  src/mesa/drivers/dri/i965/intel_debug.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/intel_debug.c b/src/mesa/drivers/dri/i965/intel_debug.c
> index a077731..b3b3c21 100644
> --- a/src/mesa/drivers/dri/i965/intel_debug.c
> +++ b/src/mesa/drivers/dri/i965/intel_debug.c
> @@ -68,7 +68,7 @@ static const struct dri_debug_control debug_control[] = {
>     { "optimizer",   DEBUG_OPTIMIZER },
>     { "ann",         DEBUG_ANNOTATION },
>     { "no8",         DEBUG_NO8 },
> -   { "vec4vs",      DEBUG_VEC4VS },
> +   { "vec4",        DEBUG_VEC4VS },
>     { "spill",       DEBUG_SPILL },
>     { "cs",          DEBUG_CS },
>     { NULL,    0 }
> --
> 2.5.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev