<div dir="ltr">On 9 August 2013 14:56, Anuj Phogat <span dir="ltr"><<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">A lot of classes and functions in ext_framebuffer_multisample/common.cpp<br>
can be utilized to develop new piglit test cases. This patch moves the<br>
Fbo and test_pattern classes to a global location tests/util.<br>
Functions from these classes will be utilized in my later patches for<br>
tests in spec/arb_framebuffer_object.<br>
<br>
Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com" target="_blank">anuj.phogat@gmail.com</a>><br></blockquote><div><br></div><div>Thanks for taking the time to do this, Anuj.  I have just one small concern: when we were talking about this in person last week, I seem to remember us deciding that the classes should either be renamed to give them a prefix, or put in a new namespace.  Does that seem reasonable to you?  I'm worried about some of the shorter class names (like "Fbo") conflicting with variables and classes we try to make in the future.  I have a minor preference for using a namespace rather than a prefix, mostly because I think it will be easier, but I could go either way.<br>
<br>With that addressed, this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br><br>I didn't get a chance to read through patches 2 and 3 in detail, but I verified that they pass on my NVIDIA system, and I confirmed visually that they appear to be doing the right thing, so consider them:<br>
<br></div><div>Acked-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div></div></div></div>