[Mesa-dev] [PATCH v2 2/6] glapi: Alias ClampColor and ClampColorARB.

Paul Berry stereotype441 at gmail.com
Mon Oct 22 17:22:42 PDT 2012


There's no reason to have separate slots in the dispatch table for
these two functions, since they are synonymous.

Reviewed-by: Brian Paul <brianp at vmware.com>
---
 src/mapi/glapi/gen/GL3x.xml | 2 +-
 src/mesa/main/api_exec.c    | 1 -
 src/mesa/main/dlist.c       | 1 -
 3 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/mapi/glapi/gen/GL3x.xml b/src/mapi/glapi/gen/GL3x.xml
index 7642a27..db8bae5 100644
--- a/src/mapi/glapi/gen/GL3x.xml
+++ b/src/mapi/glapi/gen/GL3x.xml
@@ -150,7 +150,7 @@
       <return type="const GLubyte *"/>
   </function>
 
-  <function name="ClampColor" offset="assign">
+  <function name="ClampColor" alias="ClampColorARB">
     <param name="target" type="GLenum"/>
     <param name="clamp" type="GLenum"/>
   </function>
diff --git a/src/mesa/main/api_exec.c b/src/mesa/main/api_exec.c
index d2f14ce..e2a503d 100644
--- a/src/mesa/main/api_exec.c
+++ b/src/mesa/main/api_exec.c
@@ -768,7 +768,6 @@ _mesa_create_exec_table(struct gl_context *ctx)
       SET_ClearBufferfv(exec, _mesa_ClearBufferfv);
       SET_ClearBufferfi(exec, _mesa_ClearBufferfi);
       SET_GetStringi(exec, _mesa_GetStringi);
-      SET_ClampColor(exec, _mesa_ClampColorARB);
    }
 
    /* GL_ARB_instanced_arrays */
diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c
index 28cd52a..1948bff 100644
--- a/src/mesa/main/dlist.c
+++ b/src/mesa/main/dlist.c
@@ -10143,7 +10143,6 @@ _mesa_create_save_table(const struct gl_context *ctx)
 
    /* GL_ARB_color_buffer_float */
    SET_ClampColorARB(table, save_ClampColorARB);
-   SET_ClampColor(table, save_ClampColorARB);
 
    /* GL 3.0 */
    SET_ClearBufferiv(table, save_ClearBufferiv);
-- 
1.7.12.4



More information about the mesa-dev mailing list