On 4 May 2012 11:58, Eric Anholt <span dir="ltr">&lt;<a href="mailto:eric@anholt.net" target="_blank">eric@anholt.net</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 Wed,  2 May 2012 14:30:04 -0700, Paul Berry &lt;<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>&gt; wrote:<br>
&gt; This patch adds a new test, &quot;depthstencil-render-miplevels&quot;, which<br>
&gt; verifies that we can render to all miplevels of depth and stencil<br>
&gt; textures.  It exposes two bugs in Mesa&#39;s current i965 driver:<br>
<br>
</div>Piglit is generally a C project. I&#39;d rather see the test moved to a .c<br>
file and the anonymous namespace dropped.<br>
</blockquote></div><br>Can you elaborate on what the benefit would be of this change?  Granted, Piglit is mostly C, but it has some C++ code as well.  Changing this source file from C++ to C would mean that in the future, if we wanted to use C++ features in this test, or interface this code with some of Piglit&#39;s other C++ code, we would have to switch it back to C++, or do some otherwise unnecessary hack, like placing the C++-using functions in a separate file.  Although we&#39;re unlikely to need to make use of C++ in this test in the future, it seems better to stay flexible, all other things being equal.<br>
<br>