<div dir="ltr">On 26 February 2013 02:18, Chris Forbes <span dir="ltr"><<a href="mailto:chrisf@ijw.co.nz" target="_blank">chrisf@ijw.co.nz</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
This series adds an initial set of tests for the ARB_texture_multisample<br>
extension. I will follow this with more tests to explore some edge cases.<br>
<br>
Changes from V2:<br>
    - Leftover window size junk removed<br>
    - Misc small tidyups<br>
<br>
-- Chris<br></blockquote><div><br></div><div>Chris and I have been having some discussion about this over IRC.  Just to keep the list updated:<br><br></div><div>- Patches introducing new files (2/8, 3/8, 4/8, 5/8, 8/8) need to have copyright notices added to them.  I recommended that Chris just copy and pasting one of the Intel copyright notices (e.g. the one in tests/spec/ext_transform_feedback/structs.c) and replace "Intel Corporation" with his name.<br>
<br></div><div>- When I reviewed an older version of this series on 1/7, I mentioned some other tests I would like to see added (see <a href="http://lists.freedesktop.org/archives/piglit/2013-January/004497.html">http://lists.freedesktop.org/archives/piglit/2013-January/004497.html</a>).  I'd still like to see those tests, but I don't think that we need to hold up this patch series (or landing the Mesa implementation) to wait for them.<br>
<br></div><div>- I don't feel qualified to review patch 7/8 (arb_texture_multisample: add new targets to texelFetch) so I've asked Ken if he can help.<br><br></div><div>- The tests introduced by patch 7/8 currently fail on both Chris's and my nVidia machines.  We haven't figured out why.  It's possible that it's an nVidia bug, but I think we need to get to the bottom of it before we can have confidence that the test is correct.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</blockquote></div><br></div></div>