<div dir="ltr">On 4 October 2013 13:49, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.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"><div class="im">On 10/04/2013 01:42 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 20 September 2013 12:38, Chad Versace <<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>><u></u>wrote:<br>
</blockquote>
<br>
</div><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It's not obvious clear why EGL_KHR_surfaceless_context is necessary for<br>
this test.  Can we remove this check, or add a comment explaining why it's<br>
necessary?<br>
<br>
With that addressed, this patch is:<br>
<br>
Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>><br>
</blockquote>
<br></div>
Does this comment suffice?<br>
<br>
<br>
        /* This test tries to be window-system independent, and so avoids<br>
         * creating any EGLSurface. To call eglMakeCurrent() without a surface<br>
         * requires EGL_KHR_surfaceless_context.<br>
         */<br>
        if (!piglit_is_egl_extension_<u></u>supported(dpy, "EGL_KHR_surfaceless_context")<u></u>)<br>
                piglit_report_result(PIGLIT_<u></u>SKIP);<br>
<br>
</blockquote></div><br></div><div class="gmail_extra">Ah, ok.  That explains it.  Thanks.<br></div></div>