dispatch: properly handle parameter name mismatches in glapitemp.h.
authorPaul Berry <stereotype441@gmail.com>
Tue, 30 Oct 2012 15:45:29 +0000 (08:45 -0700)
committerPaul Berry <stereotype441@gmail.com>
Thu, 1 Nov 2012 18:22:49 +0000 (11:22 -0700)
commit571d5c353a6f801c7a6ce2dd323152e5a2dbf052
tree68691f46eb6214ebf53fe8c3317a93978e0d2c32
parent33e000472080af2a48d9753847469789100fa613
dispatch: properly handle parameter name mismatches in glapitemp.h.

Previously, when code-generating aliased functions in glapitemp.h, we
weren't consistent about which function alias we used to obtain the
parameter names, with the risk that we would generate incorrect code
like this:

  KEYWORD1 void KEYWORD2 NAME(Foo)(GLint x)
  {
    (void) x;
    DISPATCH(Foo, (x), (F, "glFoo(%d);\n", x));
  }
  KEYWORD1 void KEYWORD2 NAME(FooEXT)(GLint y)
  {
    (void) x;
    DISPATCH(Foo, (x), (F, "glFooEXT(%d);\n", x));
  }

At the moment there are no aliased functions with mismatched parameter
names, so this isn't the problem.  But when we introduce GLES1
functions into the dispatch table, there will be
(MapBufferRange/MapBufferRangeEXT).  This patch paves the way for that
by fixing the code generation script to handle the mismatch correctly.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
src/mapi/glapi/gen/gl_XML.py
src/mapi/glapi/gen/gl_apitemp.py