[Mesa-dev] [PATCH 2/2] glapi: Add (empty) glapi for ARB_shader_precision

Ian Romanick idr at freedesktop.org
Thu Jan 1 22:42:40 PST 2015


If an extension doesn't add any enums or functions, I don't think we
usually bother adding it, but meh.  One nit below...


On 12/31/2014 12:43 PM, Micah Fedke wrote:
> ---
>  src/mapi/glapi/gen/gl_API.xml | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
> index e1b1246..d7d0627 100644
> --- a/src/mapi/glapi/gen/gl_API.xml
> +++ b/src/mapi/glapi/gen/gl_API.xml
> @@ -8247,7 +8247,11 @@
>  
>  <xi:include href="ARB_separate_shader_objects.xml" xmlns:xi="http://www.w3.org/2001/XInclude"/>
>  
> -<!-- ARB extensions #98...#108 -->
> +<category name="GL_ARB_shader_precision" number="98">
> +      <!-- No new functions, types, enums. -->
> +   </category>

This should be at the same indentation as the opening <category>.  With
that fixed,

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

> +
> +<!-- ARB extensions #99...#108 -->
>  
>  <xi:include href="ARB_ES2_compatibility.xml" xmlns:xi="http://www.w3.org/2001/XInclude"/>
>  
> 



More information about the mesa-dev mailing list