<div dir="ltr">On 11 July 2013 12:06, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.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 Mon, Jul 8, 2013 at 10:40 AM, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br>
> +   const glsl_type *typ(const char *name)<br>
<br>
</div>This must not be a typo, since it's consistent.<br></blockquote><div><br></div><div>Heh, "type" is a reserved word in so many languages that I guess I assumed it was reserved in C++ without even checking.  Turns out it isn't.<br>
<br></div><div>I'm doing a minor re-spin of this patch to address Ian's comments; I'll rename it to "type" in v2.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
The series is Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
<br>
with the caveat that the third patch is hard to review and I could<br>
easily have missed something, since it's nearly a total rewrite. But<br>
there's no question that the final result is better. Thanks for doing<br>
this clean up.<br>
</blockquote></div><br></div></div>