mesa/clear: simplify ClearBufferiv error handling
authorIlia Mirkin <imirkin@alum.mit.edu>
Fri, 12 Feb 2016 17:54:42 +0000 (12:54 -0500)
committerIlia Mirkin <imirkin@alum.mit.edu>
Fri, 12 Feb 2016 23:22:49 +0000 (18:22 -0500)
Might as well handle everything in the same error call.

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Brian Paul <brianp@vmware.com>
src/mesa/main/clear.c

index 5c40a03c9c4732080da84ca6e86c9af41cddb04a..568f600a55e55fc183fd1f045d9fb039b70ebeaf 100644 (file)
@@ -325,18 +325,6 @@ _mesa_ClearBufferiv(GLenum buffer, GLint drawbuffer, const GLint *value)
       _mesa_update_state( ctx );
    }
 
-   /* Page 498 of the PDF, section '17.4.3.1 Clearing Individual Buffers'
-    * of the OpenGL 4.5 spec states:
-    *
-    *    "An INVALID_ENUM error is generated by ClearBufferiv and
-    *     ClearNamedFramebufferiv if buffer is not COLOR or STENCIL."
-    */
-   if (buffer == GL_DEPTH || buffer == GL_DEPTH_STENCIL) {
-      _mesa_error(ctx, GL_INVALID_ENUM,
-                  "glClearBufferiv(buffer=GL_DEPTH || GL_DEPTH_STENCIL)");
-      return;
-   }
-
    switch (buffer) {
    case GL_STENCIL:
       /* Page 264 (page 280 of the PDF) of the OpenGL 3.0 spec says:
@@ -387,6 +375,12 @@ _mesa_ClearBufferiv(GLenum buffer, GLint drawbuffer, const GLint *value)
       }
       break;
    default:
+      /* Page 498 of the PDF, section '17.4.3.1 Clearing Individual Buffers'
+       * of the OpenGL 4.5 spec states:
+       *
+       *    "An INVALID_ENUM error is generated by ClearBufferiv and
+       *     ClearNamedFramebufferiv if buffer is not COLOR or STENCIL."
+       */
       _mesa_error(ctx, GL_INVALID_ENUM, "glClearBufferiv(buffer=%s)",
                   _mesa_enum_to_string(buffer));
       return;