On 31 October 2012 15:27, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 10/30/2012 10:42 AM, Paul Berry wrote:<br>
> This patch creates a header querymatrix.h, to allow functions defined<br>
> in querymatrix.c to be used from other .c files.  It also switches<br>
> from the nonstandard GL_APIENTRY to GLAPIENTRY.<br>
> ---<br>
>  src/mesa/main/querymatrix.c | 12 +++++-------<br>
>  src/mesa/main/querymatrix.h | 39 +++++++++++++++++++++++++++++++++++++++<br>
>  2 files changed, 44 insertions(+), 7 deletions(-)<br>
>  create mode 100644 src/mesa/main/querymatrix.h<br>
><br>
> diff --git a/src/mesa/main/querymatrix.c b/src/mesa/main/querymatrix.c<br>
> index 2843d55..27842ae 100644<br>
<br>
</div><div class="im">> +extern void GLAPIENTRY _mesa_GetIntegerv(GLenum pname, GLint *params);<br>
> +extern void GLAPIENTRY _mesa_GetFloatv(GLenum pname, GLfloat *params);<br>
<br>
</div>I think the local declarations of the Get functions should be removed. Instead,<br>
just #include "main/get.h".<br>
</blockquote></div><br>Yeah, that seems reasonable.  I'll make that change before pushing the series.<br>