<div dir="ltr">On 14 February 2013 10:27, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</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">
Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> writes:<br>
<br>
Wow, I was surprised to find that we parse our config blocks in python<br>
and pass that as arguments to be parsed by glslparsertest, rather than<br>
just parsing the config block once in C.<br>
<br>
This series looks good to me.  The one comment I'd have is to put the<br>
ARB_gs4 decision rationale in the patch 6/6 commit msg, to record why<br>
the choice was made.<br></blockquote><div><br></div><div>That's a good idea.  Will do.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
This series is:<br>
<br>
Reviewed-by: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>><br>
</blockquote></div><br>Thanks!<br></div></div>