On 19 April 2012 08:00, 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="im">On 19 April 2012 06:41, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br></div><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On Thu, Apr 19, 2012 at 12:09 AM, Paul Berry <span dir="ltr"><<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>></span> wrote:<br></div></blockquote></div><div class="im"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="gmail_quote"><div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Everyone: I'm interested in hearing<br>
opinions about whether this test is too big, and if so, how to<br>
effectively split it up into smaller tests.<br></blockquote><div><br></div></div><div>This is a fairly comprehensive test to measure the accuracy of MSAA in FBO.</div><div>I liked the way you are generating reference image. In my opinion there are lot</div>
<div>of functions in this test which could be utilized by upcoming multisample test </div><div>cases. e.g. the current piglit test I'm working on (Test switching MSAA on/off in</div><div>FBO), can utilize draw_reference_image() and measure_accuracy() functions to</div>
<div>verify if MSAA is actually switched ON in FBO. Every new multisample test case</div><div>would require these functions to verify if MSAA is happening as expected.</div><div>So, it would be helpful to split it in to two files: common.cpp, accuracy.cpp</div>
</div></blockquote></div><div><br>That's a good thought. I'll try to figure out a sensible way to factor out the reusable stuff into a separate file.<br></div></div>
</blockquote></div><br>After further reflection, it's not obvious to me how to refactor out the reusable parts of the test without a better understanding of how other tests will re-use the functionality. So I think I'd prefer to push the code as is, and then refactor it incrementally as we discover what parts are useful to other tests. If there are no objections to that, I'll go ahead and push the code tomorrow morning.<br>