On 6 October 2011 14:20, Ian Romanick <span dir="ltr">&lt;<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>&gt;</span> wrote:<br><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 10/03/2011 04:19 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This test is similar to the existing<br>
&quot;vs-clip-distance-enables.<u></u>shader_test&quot;, except that it tests the<br>
behavior of gl_ClipVertex instead of gl_ClipDistance.  Because of<br>
differences in the way gl_ClipVertex and gl_ClipDistance are<br>
implemented in Mesa, it&#39;s possible that one of these tests might pass<br>
while the other fails.<br>
</blockquote>
<br></div>
There is one thing that I don&#39;t think I noticed when vs-clip-distance-enables.<u></u>shader_test went around for review.  If the test fails in, say, the first draw will be difficult to debug because it&#39;s overdrawn by later operations.  Other than breaking this into several separate tests, I don&#39;t see a good way to fix that.<br>
</blockquote><div><br>Yeah, good point.  It would be cool if we could add a &quot;pause&quot; command to the shader_test that causes the window to be displayed and then waits for a keypress (except in &quot;-auto&quot; mode, of course, when it would just merrily continue along).  Then we could sprinkle &quot;pause&quot; commands into tests like this for easier debugging.  Hmm, that might not be that hard to implement.  I&#39;ll add it to my &quot;back burner&quot; list.<br>
<br>Another shader_runner feature that would be cool would be a &quot;-step&quot; command-line option that causes it to execute the commands one at a time, printing each command as it does so, and waiting for a keypress before it moves on to the next command.<br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
So, other than just wanting to complain, the series is<br>
<br>
Reviewed-by: Ian Romanick &lt;<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a>&gt;</blockquote><div><br>Thanks!<br></div></div>