[Mesa-dev] [PATCH] mesa: add missing DebugMessageControl types

[Timothy Arceri]

Hi Carl,

In the end I created a new series [1] to fix this issue and at the same
time fix a few other issues such as making ARB_debug_output an alias of
KHR_debug. I didn't send this to stable as I wasn't sure if this change
would be considered to invasive for stable, but it would be good in my
opinion to apply the whole series if others don't have a problem with
it. I will send the series to the stable list for consideration.

Tim

http://lists.freedesktop.org/archives/mesa-dev/2014-March/055173.html

On Tue, 2014-04-15 at 17:19 -0700, Carl Worth wrote:
> Timothy Arceri <t_arceri at yahoo.com.au> writes:
> > Forgot to CC stable. This should apply to both 10 branches.
> 
> Hi Timothy,
> 
> The specific patch you were referring to didn't get applied. But a very
> similar version did appear later (which I didn't find in my email
> archives). See below.
> 
> Shall I pick this over to both the 10.0 and the 10.1 branches?
> 
> -Carl
> 
> commit 6c3f5abc2dc99a904994fb41d6e48527ddcdf7de
> Author: Timothy Arceri <t_arceri at yahoo.com.au>
> Date:   Sat Mar 8 15:37:46 2014 +1100
> 
>     mesa: add missing DebugMessageControl types
>     
>     Signed-off-by: Timothy Arceri <t_arceri at yahoo.com.au>
>     Reviewed-by: Brian Paul <brianp at vmware.com>
> 
> diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
> index 91f62da..8ec6a8c 100644
> --- a/src/mesa/main/errors.c
> +++ b/src/mesa/main/errors.c
> @@ -524,6 +524,8 @@ validate_params(struct gl_context *ctx, unsigned caller,
>     case GL_DEBUG_TYPE_OTHER_ARB:
>     case GL_DEBUG_TYPE_MARKER:
>        break;
> +   case GL_DEBUG_TYPE_PUSH_GROUP:
> +   case GL_DEBUG_TYPE_POP_GROUP:
>     case GL_DONT_CARE:
>        if (caller == CONTROL)
>           break;