<div class="gmail_quote">On 2 March 2012 18:39, Eric Anholt <span dir="ltr"><<a href="mailto:eric@anholt.net">eric@anholt.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div class="im">On Fri, 2 Mar 2012 15:40:20 -0800, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote:<br> > Several tests were checking for the presence of GL extensions using<br> > GLEW macros. This patch changes the tests to use<br> > piglit_is_extension_supported(), which provides the same functionality<br> > in a way that is not dependent on GLEW.<br> <br> </div><div class="im">> diff --git a/tests/fbo/fbo-clear-formats.c b/tests/fbo/fbo-clear-formats.c<br> > index df0d854..ab41c45 100644<br> > --- a/tests/fbo/fbo-clear-formats.c<br> > +++ b/tests/fbo/fbo-clear-formats.c<br> > @@ -271,7 +271,7 @@ test_mipmap_drawing(int x, int y, int dim, int level, GLuint internalformat)<br> > if (compressed && dim < 8)<br> > return GL_TRUE;<br> ><br> > - if (GLEW_ARB_depth_texture) {<br> > + if (piglit_is_extension_supported("ARB_depth_texture")) {<br> > glGetTexLevelParameteriv(GL_TEXTURE_2D, level,<br> > GL_TEXTURE_DEPTH_SIZE, &d_size);<br> <br> </div>The other callers of piglit_is_extension_supported() ask for<br> GL_ARB_whatever instead of ARB_whatever. Seems like we should stick<br> with that.<br> </blockquote></div><br><div>Oh yeah. You're right. I'll fix that.</div>