[Mesa-dev] [PATCH 3/5] Bug fix: skip padding in get_called_parameter_string
Kenneth Graunke
kenneth at whitecape.org
Thu Feb 28 01:16:58 PST 2013
On 02/27/2013 04:58 PM, Eric Anholt wrote:
> From: Paul Berry <stereotype441 at gmail.com>
>
> This bug is currently benign, since get_called_parameter_string() is
> currently only used for functions that return true for
> glx_function.has_different_protocol(), and none of those functions
> include padding. However, in order to implement marshalling of GL API
> functions, we'll need to use get_called_parameter_string() far more
> often.
> ---
> src/mapi/glapi/gen/gl_XML.py | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/src/mapi/glapi/gen/gl_XML.py b/src/mapi/glapi/gen/gl_XML.py
> index 227e9fc..3bbc794 100644
> --- a/src/mapi/glapi/gen/gl_XML.py
> +++ b/src/mapi/glapi/gen/gl_XML.py
> @@ -802,6 +802,8 @@ class gl_function( gl_item ):
> comma = ""
>
> for p in self.parameterIterator():
> + if p.is_padding:
> + continue
> p_string = p_string + comma + p.name
> comma = ", "
Patches 1-3 are:
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
but need a proper prefix in the commit message (i.e. "mesa:")
I don't know how the counter mechanism works, so I don't feel qualified
to review 4 and 5.
More information about the mesa-dev
mailing list