mesa: accept TYPE_PUSH/POP_GROUP with glDebugMessageInsert
authorEmil Velikov <emil.l.velikov@gmail.com>
Thu, 26 Nov 2015 00:36:15 +0000 (00:36 +0000)
committerEmil Velikov <emil.l.velikov@gmail.com>
Thu, 3 Dec 2015 19:21:08 +0000 (19:21 +0000)
These new (relative to ARB_debug_output) tokens, have been explicitly
separated from the existing ones in the spec text. With the reference
to glDebugMessageInsert was dropped.

At the same time, further down the spec says:
   "The value of <type> must be one of the values from Table 5.4"

... and these two are listed in Table 5.4.

The GL 4.3 and GLES 3.2 do not give any hints on the former
'definition', plus CTS requires that the tokens are valid values for
glDebugMessageInsert.

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Reviewed-by: Timothy Arceri <timothy.arceri@collabora.com>
src/mesa/main/errors.c

index a24700072d8b6f073e77485560de409971e34ad9..9fa2c497a2eb56d4be3d5d678decb6e77c629958 100644 (file)
@@ -922,9 +922,9 @@ validate_params(struct gl_context *ctx, unsigned caller,
    case GL_DEBUG_TYPE_PORTABILITY_ARB:
    case GL_DEBUG_TYPE_OTHER_ARB:
    case GL_DEBUG_TYPE_MARKER:
-      break;
    case GL_DEBUG_TYPE_PUSH_GROUP:
    case GL_DEBUG_TYPE_POP_GROUP:
+      break;
    case GL_DONT_CARE:
       if (caller == CONTROL)
          break;