Mesa (master): mesa: add GL_DEPTH_STENCIL to glCopyPixels error check

Brian Paul brianp at kemper.freedesktop.org
Thu Aug 20 14:12:58 UTC 2009


Module: Mesa
Branch: master
Commit: 39ec4a9253197c4507d53ce9a3c1d8963d8f5cea
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=39ec4a9253197c4507d53ce9a3c1d8963d8f5cea

Author: Brian Paul <brianp at vmware.com>
Date:   Thu Aug 20 08:10:14 2009 -0600

mesa: add GL_DEPTH_STENCIL to glCopyPixels error check

Plus, add a comment about 'type' error checking.
See bug 19087.

---

 src/mesa/main/drawpix.c |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)

diff --git a/src/mesa/main/drawpix.c b/src/mesa/main/drawpix.c
index 6d31f32..67311f7 100644
--- a/src/mesa/main/drawpix.c
+++ b/src/mesa/main/drawpix.c
@@ -152,7 +152,14 @@ _mesa_CopyPixels( GLint srcx, GLint srcy, GLsizei width, GLsizei height,
       return;
    }
 
-   if (type != GL_COLOR && type != GL_DEPTH && type != GL_STENCIL) {
+   /* Note: more detailed 'type' checking is done by the
+    * _mesa_source/dest_buffer_exists() calls below.  That's where we
+    * check if the stencil buffer exists, etc.
+    */
+   if (type != GL_COLOR &&
+       type != GL_DEPTH &&
+       type != GL_STENCIL &&
+       type != GL_DEPTH_STENCIL) {
       _mesa_error(ctx, GL_INVALID_ENUM, "glCopyPixels(type=%s)",
                   _mesa_lookup_enum_by_nr(type));
       return;




More information about the mesa-commit mailing list