mesa: change GL_ALL_ATTRIB_BITS to 0xFFFFFFFF
authorBrian Paul <brianp@vmware.com>
Sat, 1 Feb 2014 17:51:43 +0000 (10:51 -0700)
committerBrian Paul <brianp@vmware.com>
Mon, 3 Feb 2014 19:50:03 +0000 (12:50 -0700)
This has been wrong for many years.  It was originally 0x000FFFFF and long
ago there was discussion about whether GL_ALL_ATTRIB_BITS should include
the then-new GL_MULTISAMPLE_BIT bit.  Eventually the ARB decided that
glPushAttrib(GL_ALL_ATTRIB_BITS) should save all current and future
attribute groups (hence ~0).  Unfortunately, Mesa's gl.h was never updated.

This was just recently spotted by Eric Anholt and reported as a bug to the
ARB.  Ian, Jon Leech and I discussed it at the ARB meeting and decided to
change Mesa's value to reflect the ARB's decision.

Acked-by: Eric Anholt <eric@anholt.net>
include/GL/gl.h

index 48343f65d93feb094d26d0d0977369b646b42c22..4e2932df7df44db834fd3636f12d5b5692438ec5 100644 (file)
@@ -694,7 +694,7 @@ typedef double              GLclampd;       /* double precision float in [0,1] */
 #define GL_LIST_BIT                            0x00020000
 #define GL_TEXTURE_BIT                         0x00040000
 #define GL_SCISSOR_BIT                         0x00080000
-#define GL_ALL_ATTRIB_BITS                     0x000FFFFF
+#define GL_ALL_ATTRIB_BITS                     0xFFFFFFFF
 
 
 /* OpenGL 1.1 */