On 3 January 2012 18:56, Eric Anholt <span dir="ltr">&lt;<a href="mailto:eric@anholt.net">eric@anholt.net</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="HOEnZb"><div class="h5">On Tue, 3 Jan 2012 18:51:24 -0800, Paul Berry &lt;<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>&gt; wrote:<br>
&gt; On 3 January 2012 18:16, Eric Anholt &lt;<a href="mailto:eric@anholt.net">eric@anholt.net</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt;<br>
&gt; &gt; This series is Reviewed-by: Eric Anholt &lt;<a href="mailto:eric@anholt.net">eric@anholt.net</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt; With this and the fix I have for glGetTransformFeedbackVarying(), I<br>
&gt; &gt; think we should be passing the oglc tests, except for one more case:<br>
&gt; &gt; They try to ask for whole arrays to be fed back, without [] in the<br>
&gt; &gt; declaration.  The clearest text I could find on this point was a<br>
&gt; &gt; RESOLUTION: in the spec, but it kind of sounded to me like the<br>
&gt; &gt; resolution was about working around &quot;how to I get feedback from my huge<br>
&gt; &gt; amount of varying data when I have so few TFB attributes available?&quot;<br>
&gt; &gt; Did you end up testing whether other drivers accepted non-subscripted<br>
&gt; &gt; TFB varyings for varying arrays?<br>
&gt; &gt;<br>
&gt;<br>
&gt; No, I didn&#39;t write any tests of non-subscripted arrays.  My interpretation<br>
&gt; of the spec had been that they weren&#39;t allowed, but now that I&#39;m re-reading<br>
&gt; it, I&#39;m reconsidering.  I&#39;ll investigate on the nVidia proprietary linux<br>
&gt; driver and let you know what I find out.<br>
&gt;<br>
&gt; Do you have Piglit tests for glGetTransformFeedbackVarying()?  I was<br>
&gt; starting to write some, but it was slow going and I don&#39;t want to duplicate<br>
&gt; effort.<br>
<br>
</div></div>Nope, I was meaning to get at least a test of the particular piece of<br>
spec I was hung up on (GetTFVarying reflecting linked state, not next<br>
state), and didn&#39;t get to it.<br>
</blockquote></div><br>Ok.  Since I&#39;ve spent the last two weeks or so writing tests for code other people wrote, would you mind writing the Piglit tests for GetTransformFeedbackVarying?  That would have the side benefit of giving me time to respond to your comments on my other Piglit tests that are still in flight.<br>