<div dir="ltr">On 26 August 2013 16:53, 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">
<div class="HOEnZb"><div class="h5">On 08/26/2013 04:14 PM, Eric Anholt wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ian Romanick <<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>> writes:<br>
<br>
<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" target="_blank">ian.d.romanick@intel.com</a>><br>
<br>
Instead of replicating the same code over and over and over, just put it<br>
in one file that each shader test can reference.<br>
</blockquote>
<br>
I've been meaning to kill "[vertex/fragment shader file]" support from<br>
shader_runner for a while. :(<br>
<br>
What I don't like about it is that it scatters the information about a<br>
particular test to more places, which shader_runner had helped reduce<br>
significantly.  And this feature is used very rarely -- looking at the<br>
current list of tests using it, they mostly look like the first few<br>
shader_runner tests we made.<br>
<br>
For the changes you wanted to do in this series, a sed job looks like it<br>
would have been just fine.<br>
</blockquote>
<br></div></div>
I think it's not used much because most people don't know it's available.  These are the (only?) cases where it's useful: you don't want to type the dummy, pass-through vertex shader over and over and over.  Maybe we should replace it with a more limited "[vertex shader passthrough]" option?<br>
</blockquote><div><br></div>FWIW the reason I don't use it isn't because I don't know it's available.  It's because when writing a test, it feels easier for me to just pound out the dummy pass-through vertex shader rather than to go hunt around nearby directories to find the shader I want.  I also find it slightly easier to code review others' shader tests if they don't use this feature, because it means I can completely understand the shader test in isolation, and I don't feel compelled to double-check that the file they're referring to actually exists (and does what it claims).<br>
<br></div><div class="gmail_quote">I like the idea of a "[vertex shader passthrough]" option--it would neatly address both of the reasons why I don't currently use [vertex shader file].<br><br></div><div class="gmail_quote">
However, I don't have terribly strong opinions about this stuff.  If you want to just leave the series as is, that's fine by me as well.<br></div></div></div>