On 3 January 2012 18:51, Paul Berry <span dir="ltr"><<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</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="HOEnZb"><div class="h5">On 3 January 2012 18:16, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</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">
<br>
This series is Reviewed-by: Eric Anholt <<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</a>><br>
<br>
With this and the fix I have for glGetTransformFeedbackVarying(), I<br>
think we should be passing the oglc tests, except for one more case:<br>
They try to ask for whole arrays to be fed back, without [] in the<br>
declaration. The clearest text I could find on this point was a<br>
RESOLUTION: in the spec, but it kind of sounded to me like the<br>
resolution was about working around "how to I get feedback from my huge<br>
amount of varying data when I have so few TFB attributes available?"<br>
Did you end up testing whether other drivers accepted non-subscripted<br>
TFB varyings for varying arrays?<br>
</blockquote></div><br></div></div>No, I didn't write any tests of non-subscripted arrays. My interpretation of the spec had been that they weren't allowed, but now that I'm re-reading it, I'm reconsidering. I'll investigate on the nVidia proprietary linux driver and let you know what I find out.<br>
</blockquote><div><br>Argh, nVidia supports non-subscripted arrays just fine, even on gl_ClipDistance.<br><br>I had been hoping we wouldn't have to deal with this, based on the same RESOLUTION you saw. But in retrospect that was wishful thinking. It's clear from the rest of the EXT_transform_feedback spec that non-subscripted arrays have always been intended to work. The clincher is the "size" argument of GetTransformFeedbackVarying(), which would be pointless if non-subscripted arrays weren't allowed.<br>
<br>I feel responsible for this bug since my wishful thinking prevented it from being discovered earlier. I'll get a fix, and some tests, out to the list ASAP.<br></div></div>