On Wed, Jun 5, 2013 at 10:13 AM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</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">Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>> writes:<br>
<br>
> On 06/05/2013 03:53 AM, Arnas Milasevicius wrote:<br>
>><br>
</div><div class="im">>> diff --git a/src/mapi/glapi/gen/gl_enums.py b/src/mapi/glapi/gen/gl_enums.py<br>
>> index 806d384..ba19196 100644<br>
>> --- a/src/mapi/glapi/gen/gl_enums.py<br>
>> +++ b/src/mapi/glapi/gen/gl_enums.py<br>
>> @@ -91,7 +91,7 @@ static int compar_nr( const int *a, const unsigned *b )<br>
>><br>
>>   static char token_tmp[20];<br>
>><br>
>> -const char *_mesa_lookup_enum_by_nr( int nr )<br>
>> +const char *_mesa_enum_nr( int nr )<br>
><br>
> I don't think this is a good name for the function.  I'd suggest<br>
> _mesa_enum_string() since we're returning the string for the given enum<br>
> value, not a number.<br>
<br>
</div>_mesa_get_enum_name() to match _mesa_get_format_name()?<br>
<br><br></blockquote><div>Sure, that's good too.<br><br>-Brian<br> <br></div></div><br>