<div dir="ltr">On 16 December 2013 14:42, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br><div class="gmail_extra"><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">On 12/16/2013 03:27 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
These enums were redundant.<br>
---<br>
  src/glsl/ast_to_hir.cpp         | 82 ++++++++++++++++++++----------<u></u>-----------<br>
  src/glsl/builtin_functions.cpp  | 22 +++++------<br>
  src/glsl/builtin_variables.cpp  | 20 +++++-----<br>
  src/glsl/glsl_parser.yy         |  8 ++--<br>
  src/glsl/glsl_parser_extras.<u></u>cpp | 14 +++----<br>
  src/glsl/glsl_parser_extras.h   | 10 +----<br>
  6 files changed, 75 insertions(+), 81 deletions(-)<br>
</blockquote>
<br></div>
For the series:<br>
Reviewed-by: Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>><br>
<br>
I see there's more MESA_SHADER_x -> string conversion in<br>
<br>
check_resources() in linker.cpp<br>
link_check_atomic_counter_<u></u>resources() in link_atomic.cpp<br>
<br>
Those should probably use _mesa_glsl_shader_target_name(<u></u>)<br></blockquote><div><br></div><div>That's a good point.  I'll submit a follow-up patch.<br></div></div></div></div>