On 6 October 2011 14:20, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org">idr@freedesktop.org</a>></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>
"vs-clip-distance-enables.<u></u>shader_test", 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'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'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's overdrawn by later operations. Other than breaking this into several separate tests, I don't see a good way to fix that.<br>
</blockquote><div><br>Yeah, good point. It would be cool if we could add a "pause" command to the shader_test that causes the window to be displayed and then waits for a keypress (except in "-auto" mode, of course, when it would just merrily continue along). Then we could sprinkle "pause" commands into tests like this for easier debugging. Hmm, that might not be that hard to implement. I'll add it to my "back burner" list.<br>
<br>Another shader_runner feature that would be cool would be a "-step" 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 <<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a>></blockquote><div><br>Thanks!<br></div></div>