On 10/17/2012 12:14 PM, Adam Jackson wrote:
On 10/17/12 2:52 PM, Brian Paul wrote:
On 10/17/2012 12:38 PM, Adam Jackson wrote:
Note: This is a candidate for the stable branches.
Signed-off-by: Adam Jackson <a...@redhat.com>
---
src/glx/glxext.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/glx/glxext.c b/src/glx/glxext.c
index 4d59a06..e55c1c1 100644
--- a/src/glx/glxext.c
+++ b/src/glx/glxext.c
@@ -81,6 +81,7 @@ static /* const */ char *error_list[] = {
"GLXBadPbuffer",
"GLXBadCurrentDrawable",
"GLXBadWindow",
+ "GLXBadProfileARB",
};
#ifdef GLX_USE_APPLEGL
Reviewed-by: Brian Paul <bri...@vmware.com>
But could we also add an assertion somewhere to catch missing strings in
the future? Like assert(ARRAY_SIZE(error_list) == __GLX_NUMBER_ERRORS)?
Pushed, thanks. I'll try to come up with a way to make that fail at
build time, the failure mode for getting this wrong ends up being
segfaulty.
- ajax
The STATIC_ASSERT macro in main/compiler.h would probably do the trick.
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
