[Mesa-dev] [PATCH 10/11] glapi/glx: Mark byteswap functions as _X_UNUSED

Ian Romanick idr at freedesktop.org
Mon Mar 28 17:55:47 UTC 2016


This patch is

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

On 03/24/2016 11:10 AM, Adam Jackson wrote:
> Squashes the one remaining warning in the xserver build.
> 
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  src/mapi/glapi/gen/glX_proto_recv.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mapi/glapi/gen/glX_proto_recv.py b/src/mapi/glapi/gen/glX_proto_recv.py
> index a138adb..7eb19df 100644
> --- a/src/mapi/glapi/gen/glX_proto_recv.py
> +++ b/src/mapi/glapi/gen/glX_proto_recv.py
> @@ -171,7 +171,7 @@ class PrintGlxDispatchFunctions(glX_proto_common.glx_print_proto):
>                  if t.glx_name not in already_done:
>                      real_name = self.real_types[t_size]
>  
> -                    print 'static %s' % (t_name)
> +                    print 'static _X_UNUSED %s' % (t_name)
>                      print 'bswap_%s( const void * src )' % (t.glx_name)

I think this is also non-conventional whitespace.  I'd be in favor of
fixing that too.

>                      print '{'
>                      print '    union { %s dst; %s ret; } x;' % (real_name, t_name)
> 



More information about the mesa-dev mailing list