On 19 April 2012 08:00, Paul Berry <span dir="ltr">&lt;<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</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="im">On 19 April 2012 06:41, Anuj Phogat <span dir="ltr">&lt;<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>&gt;</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">&lt;<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>&gt;</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&#39;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&#39;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&#39;s a good thought.  I&#39;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&#39;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&#39;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&#39;ll go ahead and push the code tomorrow morning.<br>