<div dir="ltr">On 10 September 2013 12:10, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</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">From: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com">ian.d.romanick@intel.com</a>><br>
<br>
Previously gl_constants::MaxVaryingComponents was used.  Now<br>
gl_constants::VertexProgram::MaxOutputs and<br>
gl_constants::GeometryProgram::MaxOutputs are used.<br>
<br>
This means that st_extensions.c had to be updated to set these fields<br>
instead of o MaxVaryingComponents.  It was previously the only place<br>
that set MaxVaryingComponents.<br></blockquote><div><br></div><div>To clarify, does that mean that on non-state-tracker-based drivers, these queries return uninitialized data?  I'm fine with that, since it's the case both before and after this patch (and it looks like you fix it in patch 5), but you might want to make a mention of it in the commit message.<br>
<br></div><div>Either way, the patch is:<br><br></div><div>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></div></div></div></div>