Mesa (9.0): glapi/glx: rename 'table' variable to 'disp_table'

Ian Romanick idr at kemper.freedesktop.org
Fri Sep 14 08:20:09 UTC 2012


Module: Mesa
Branch: 9.0
Commit: 8b8416676e959d2ef4ade0d52aa7792641a5ed6c
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=8b8416676e959d2ef4ade0d52aa7792641a5ed6c

Author: Brian Paul <brianp at vmware.com>
Date:   Thu Sep  6 08:16:56 2012 -0600

glapi/glx: rename 'table' variable to 'disp_table'

This fixes an issue where the local 'table' variable was hiding the
function parameter name in glGetColorTable(..., void *table).

This should be OK as long as there's never a GL entrypoint that uses
'disp_table' as a parameter name.

Note: This is a candidate for the 9.0 branch.

Reviewed-by: Jose Fonseca <jfonseca at vmware.com>
(cherry picked from commit 043f66204b1a190e18747c3befa8826c82dd87a7)

---

 src/mapi/glapi/gen/glX_proto_send.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/mapi/glapi/gen/glX_proto_send.py b/src/mapi/glapi/gen/glX_proto_send.py
index c533592..ae12c45 100644
--- a/src/mapi/glapi/gen/glX_proto_send.py
+++ b/src/mapi/glapi/gen/glX_proto_send.py
@@ -423,9 +423,9 @@ __indirect_get_proc_address(const char *name)
 				print ''
 				print '#if defined(GLX_DIRECT_RENDERING) && !defined(GLX_USE_APPLEGL)'
 				print '    if (gc->isDirect) {'
-				print '        const _glapi_proc *const table = GET_DISPATCH();'
+				print '        const _glapi_proc *const disp_table = GET_DISPATCH();'
 				print '        PFNGL%sPROC p =' % (name.upper())
-				print '            (PFNGL%sPROC) table[%d];' % (name.upper(), func.offset)
+				print '            (PFNGL%sPROC) disp_table[%d];' % (name.upper(), func.offset)
 				print '    %sp(%s);' % (ret_string, func.get_called_parameter_string())
 				print '    } else'
 				print '#endif'




More information about the mesa-commit mailing list