<div dir="ltr">Will it be approved or I need to resend this patch with a better commit massage?<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 5, 2013 at 8:47 PM, Arnas Milaševičius <span dir="ltr"><<a href="mailto:mla@arnas.me" target="_blank">mla@arnas.me</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>_mesa_lookup_enum_by_nr() -> _mesa_enum_string()<br></div>_mesa_lookup_prim_by_nr() -> _mesa_prim_string()<br>
</div>_mesa_lookup_enum_by_name() -> _mesa_enum_value()<br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Jun 5, 2013 at 8:20 PM, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>Arnas Milasevicius <<a href="mailto:giant1gf@gmail.com" target="_blank">giant1gf@gmail.com</a>> writes:<br>
<br>
> _mesa_prim_nr changed in:<br>
><br>
> src/mesa/main/api_validate.c<br>
> src/mesa/main/api_validate.c<br>
> src/mesa/main/enums.h<br>
> src/mesa/vbo/vbo_exec_draw.c<br>
> src/mesa/vbo/vbo_save_api.c<br>
> src/mesa/vbo/vbo_save_loopback.c<br>
><br>
> _mesa_enum_name changed in:<br>
><br>
> src/mesa/main/enums.h<br>
> src/mesa/main/tests/enum_strings.cpp<br>
> src/mesa/main/tests/enum_strings.cpp<br>
><br>
> _mesa_enum_nr changed in:<br>
><br>
> Pretty much in these and other files<br>
<br>
</div>I would recommend<br>
<a href="http://who-t.blogspot.de/2009/12/on-commit-messages.html" target="_blank">http://who-t.blogspot.de/2009/12/on-commit-messages.html</a> for learning<br>
how to write commit messages.  From this commit message, I don't know<br>
what the function names changed to, or why they changed.  The list of<br>
files changed is not interesting to include in a commit message, because<br>
you go look at the commit to see that.<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>