On Sat, Sep 10, 2011 at 6:39 AM, Ian Romanick <i...@freedesktop.org> wrote:
> From: Ian Romanick <ian.d.roman...@intel.com>
>
> All of the extensions actually supported by Mesa have been remapped by
> remap.c for a long time. Emitting all of these data structures is
> just clutter.
>
> Drivers that need additional functions remapped, should add
> 'offset="assign"' to the function definition in the .xml file.
>
> The changes to remap_helper.h are in a follow-on ~8700 line patch that
> would surely be rejected by the mailing list.
>
> Cc: Chia-I Wu <o...@lunarg.com>
Reviewed-by: Chia-I Wu <o...@lunarg.com>
> ---
> src/mapi/glapi/gen/remap_helper.py | 43 ++++-------------------------------
> 1 files changed, 5 insertions(+), 38 deletions(-)
>
> diff --git a/src/mapi/glapi/gen/remap_helper.py
> b/src/mapi/glapi/gen/remap_helper.py
> index 367ae24..0008634 100644
> --- a/src/mapi/glapi/gen/remap_helper.py
> +++ b/src/mapi/glapi/gen/remap_helper.py
> @@ -74,7 +74,11 @@ class PrintGlRemap(gl_XML.gl_print_base):
> pool_indices = {}
>
> print '/* this is internal to remap.c */'
> - print '#ifdef need_MESA_remap_table'
> + print '#ifndef need_MESA_remap_table'
> + print '#error Only remap.c should include this file!'
> + print '#endif /* need_MESA_remap_table */'
> + print ''
> +
> print ''
> print 'static const char _mesa_function_pool[] ='
>
> @@ -156,43 +160,6 @@ class PrintGlRemap(gl_XML.gl_print_base):
> print ' { -1, -1 }'
> print '};'
> print ''
> -
> - print '#endif /* need_MESA_remap_table */'
> - print ''
> -
> - # output remap helpers for DRI drivers
> -
> - for ext in extensions:
> - funcs = []
> - remapped = []
> - for f in extension_functions[ext]:
> - if f.assign_offset:
> - # these are handled above
> - remapped.append(f)
> - else:
> - # these functions are either in the
> - # abi, or have offset -1
> - funcs.append(f)
> -
> - print '#if defined(need_%s)' % (ext)
> - if remapped:
> - print '/* functions defined in
> MESA_remap_table_functions are excluded */'
> -
> - # output extension functions that need to be mapped
> - print 'static const struct gl_function_remap
> %s_functions[] = {' % (ext)
> - for f in funcs:
> - if f.offset >= 0:
> - print ' { %5d, _gloffset_%s },' \
> - % (pool_indices[f],
> f.name)
> - else:
> - print ' { %5d, -1 }, /* %s */' % \
> - (pool_indices[f],
> f.name)
> - print ' { -1, -1 }'
> - print '};'
> -
> - print '#endif'
> - print ''
> -
> return
>
>
> --
> 1.7.6
>
>
--
o...@lunarg.com
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
