I'm trying to transition Piglit from using GLUT to using Waffle. This
requires killing all uses of GLUT functions, one-by-one.

Only three tests used glutBitmapCharacter,
    texturing/getteximage-formats
    texturing/texture-packed-formats
    texturing/texwrap
and the test results were not affected by the function. It was used only
in non-auto mode to print information about the texture format to the
window.

These tests were removed from the build when using Waffle, since
glut_waffle does not implement glutBitmapCharacter. This patch returns
them to the build.

Signed-off-by: Chad Versace <chad.vers...@linux.intel.com>
---
 tests/texturing/CMakeLists.gl.txt        | 10 ++---
 tests/texturing/getteximage-formats.c    | 65 +++-----------------------------
 tests/texturing/texture-packed-formats.c | 63 -------------------------------
 tests/texturing/texwrap.c                | 10 -----
 4 files changed, 9 insertions(+), 139 deletions(-)

diff --git a/tests/texturing/CMakeLists.gl.txt 
b/tests/texturing/CMakeLists.gl.txt
index c439273..b9eca49 100644
--- a/tests/texturing/CMakeLists.gl.txt
+++ b/tests/texturing/CMakeLists.gl.txt
@@ -10,13 +10,6 @@ link_libraries (
        ${OPENGL_glu_LIBRARY}
 )
 
-if(NOT USE_WAFFLE)
-       # These tests use glutBitmapCharacter.
-       piglit_add_executable (getteximage-formats getteximage-formats.c)
-       piglit_add_executable (texture-packed-formats texture-packed-formats.c)
-       piglit_add_executable (texwrap texwrap.c)
-endif(NOT USE_WAFFLE)
-
 piglit_add_executable (1-1-linear-texture 1-1-linear-texture.c)
 piglit_add_executable (array-depth-roundtrip array-depth-roundtrip.c)
 piglit_add_executable (array-texture array-texture.c)
@@ -35,6 +28,7 @@ piglit_add_executable (gen-compressed-teximage 
gen-compressed-teximage.c)
 piglit_add_executable (gen-nonzero-unit gen-nonzero-unit.c)
 piglit_add_executable (gen-teximage gen-teximage.c)
 piglit_add_executable (gen-texsubimage gen-texsubimage.c)
+piglit_add_executable (getteximage-formats getteximage-formats.c)
 piglit_add_executable (getteximage-simple getteximage-simple.c)
 piglit_add_executable (getteximage-luminance getteximage-luminance.c)
 piglit_add_executable (incomplete-texture incomplete-texture.c)
@@ -73,6 +67,8 @@ piglit_add_executable (texdepth texdepth.c)
 piglit_add_executable (teximage-errors teximage-errors.c)
 piglit_add_executable (texrect-many texrect-many.c)
 piglit_add_executable (texredefine texredefine.c)
+piglit_add_executable (texture-packed-formats texture-packed-formats.c)
+piglit_add_executable (texwrap texwrap.c)
 piglit_add_executable (depth-tex-modes      depth-tex-modes.c 
depth-tex-modes-common.c)
 piglit_add_executable (depth-tex-modes-rg   depth-tex-modes-rg.c 
depth-tex-modes-common.c)
 piglit_add_executable (depth-tex-modes-glsl depth-tex-modes-glsl.c)
diff --git a/tests/texturing/getteximage-formats.c 
b/tests/texturing/getteximage-formats.c
index e48439e..a14dd4e 100644
--- a/tests/texturing/getteximage-formats.c
+++ b/tests/texturing/getteximage-formats.c
@@ -82,36 +82,6 @@ make_texture_image(GLenum intFormat, GLubyte 
upperRightTexel[4])
        return glGetError() == GL_NO_ERROR;
 }
 
-
-static void
-print(int x, int y, const char *s)
-{
-       glColor3f(1, 1, 1);
-       glWindowPos2iARB(x, y);
-       while (*s) {
-               glutBitmapCharacter(GLUT_BITMAP_8_BY_13, (int) *s);
-               s++;
-       }
-}
-
-
-static void
-draw_format_text(const struct test_desc *test,
-                const struct format_desc *fmt)
-{
-       char s[200];
-       sprintf(s, "%s: %s", test->param, fmt->name);
-       print(10, piglit_height - 20, s);
-}
-
-
-static void
-draw_unsupported(void)
-{
-       print(10, piglit_height - 40, "Not supported by test");
-}
-
-
 static GLfloat
 ubyte_to_float(GLubyte b, GLint bits)
 {
@@ -352,19 +322,12 @@ test_format(const struct test_desc *test,
 
        glClear(GL_COLOR_BUFFER_BIT);
 
-       if (!piglit_automatic) {
-               draw_format_text(test, fmt);
-       }
-
-       if (fmt->internalformat == GL_COMPRESSED_RGBA_S3TC_DXT1_EXT ||
-                fmt->internalformat == GL_COMPRESSED_SRGB_ALPHA_S3TC_DXT1_EXT) 
{
-               /* The RGBA_DXT1 formats seem to expose a Mesa/libtxc_dxtn bug.
-                * Just skip them for now.  Testing the other compressed formats
-                * is good enough.
-                */
-               draw_unsupported();
-       }
-       else {
+       /* The RGBA_DXT1 formats seem to expose a Mesa/libtxc_dxtn bug.
+        * Just skip them for now.  Testing the other compressed formats
+        * is good enough.
+        */
+       if (fmt->internalformat != GL_COMPRESSED_RGBA_S3TC_DXT1_EXT &&
+           fmt->internalformat != GL_COMPRESSED_SRGB_ALPHA_S3TC_DXT1_EXT) {
                /* init texture image */
                if (!make_texture_image(fmt->internalformat, upperRightTexel))
                        return GL_TRUE; /* unsupported = OK */
@@ -383,9 +346,6 @@ test_format(const struct test_desc *test,
                glDisable(GL_TEXTURE_2D);
                glDisable(GL_BLEND);
 
-               if (!piglit_automatic)
-                       print(x, y - 18, "Textured Quad");
-
                x += TEX_WIDTH + 20;
 
                level = 0;
@@ -405,17 +365,6 @@ test_format(const struct test_desc *test,
                        glDisable(GL_BLEND);
 
                        assert(!glIsEnabled(GL_TEXTURE_2D));
-                       if (!piglit_automatic) {
-                               if (level == 0) {
-                                       print(x, y - 18, "glGetTexImage");
-                                       print(x, y - 30, "level 0");
-                               }
-                               else {
-                                       char s[10];
-                                       sprintf(s, "%d", level);
-                                       print(x, y - 18, s);
-                               }
-                       }
 
                        if (level <= 2) {
                                GLint rx = x + w-1;
@@ -513,8 +462,6 @@ piglit_display(void)
                }
                else {
                        glClear(GL_COLOR_BUFFER_BIT);
-                       draw_format_text(set, &set->format[format_index]);
-                       draw_unsupported();
                        piglit_present_results();
                }
        }
diff --git a/tests/texturing/texture-packed-formats.c 
b/tests/texturing/texture-packed-formats.c
index 91e0a2f..a902c7a 100644
--- a/tests/texturing/texture-packed-formats.c
+++ b/tests/texturing/texture-packed-formats.c
@@ -44,9 +44,6 @@ PIGLIT_GL_TEST_MAIN(
     620 /*window_height*/,
     GLUT_RGB | GLUT_DOUBLE)
 
-static GLboolean PrintInfo = GL_TRUE;
-
-
 struct pixel_format {
    const char *name;
    GLenum format;
@@ -144,16 +141,6 @@ static GLboolean Blend = GL_FALSE;  /* XXX not used at 
this time */
 
 
 static void
-PrintString(const char *s)
-{
-   while (*s) {
-      glutBitmapCharacter(GLUT_BITMAP_8_BY_13, (int) *s);
-      s++;
-   }
-}
-
-
-static void
 MakeTexture(GLuint dims, const struct pixel_format *format,
             GLenum intFormat, GLboolean swap)
 {
@@ -261,7 +248,6 @@ Test(GLuint intFmt, GLuint dims)
 {
    static const float red[4] = {1, 0, 0, 1};
    static const float green[4] = {0, 1, 0, 1};
-   char s[1000];
    int w = 350, h = 20;
    int i, swap;
    GLboolean pass = GL_TRUE;
@@ -308,13 +294,6 @@ Test(GLuint intFmt, GLuint dims)
             glDisable(GL_TEXTURE_2D);
 
          glDisable(GL_BLEND);
-
-         if (PrintInfo) {
-            glColor3f(0, 0, 0);
-            glRasterPos2i(8, 6);
-            PrintString(Formats[i].name);
-         }
-
          glPopMatrix();
 
          /* test rendering */
@@ -332,51 +311,11 @@ Test(GLuint intFmt, GLuint dims)
       }
    }
 
-   if (PrintInfo) {
-      i = NUM_FORMATS;
-      glPushMatrix();
-      glTranslatef(2, i * (h + 2), 0);
-      glColor3f(1, 1, 1);
-      glRasterPos2i(8, 6);
-      PrintString("Normal");
-      glRasterPos2i(w + 2, 6);
-      PrintString("Byte Swapped");
-      glPopMatrix();
-
-      glPushMatrix();
-      glTranslatef(2, (i + 1) * (h + 2), 0);
-      glRasterPos2i(8, 6);
-      sprintf(s, "Internal Texture Format: %s (%d of %u)",
-              IntFormats[intFmt].name, intFmt + 1,
-              (unsigned) NUM_INT_FORMATS);
-      PrintString(s);
-      glPopMatrix();
-
-      glPushMatrix();
-      glTranslatef(2, (i + 2) * (h + 2), 0);
-      glRasterPos2i(8, 6);
-      if (dims == 3)
-         PrintString("Target: GL_TEXTURE_3D");
-      else
-         PrintString("Target: GL_TEXTURE_2D");
-      glPopMatrix();
-
-      glPushMatrix();
-      glTranslatef(2, (i + 3) * (h + 2), 0);
-      glRasterPos2i(8, 6);
-      if (Blend)
-         PrintString("Blend: Yes");
-      else
-         PrintString("Blend: No");
-      glPopMatrix();
-   }
-
    piglit_present_results();
 
    return pass;
 }
 
-
 enum piglit_result
 piglit_display(void)
 {
@@ -414,7 +353,5 @@ piglit_init(int argc, char **argv)
    glTexParameteri(GL_TEXTURE_3D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
    glTexEnvi(GL_TEXTURE_ENV, GL_TEXTURE_ENV_MODE, GL_REPLACE);
    glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA);
-
-   PrintInfo = !piglit_automatic;
 }
 
diff --git a/tests/texturing/texwrap.c b/tests/texturing/texwrap.c
index 2fb3d03..8e86d52 100644
--- a/tests/texturing/texwrap.c
+++ b/tests/texturing/texwrap.c
@@ -393,15 +393,6 @@ PIGLIT_GL_TEST_MAIN(
 
 extern int piglit_automatic;
 
-
-static void print_string(const char *s)
-{
-    while (*s) {
-        glutBitmapCharacter(GLUT_BITMAP_HELVETICA_10, (int) *s);
-        s++;
-    }
-}
-
 static void sample_nearest(int x, int y, int z,
                            GLenum wrap_mode, GLenum filter,
                            unsigned char pixel[4])
@@ -724,7 +715,6 @@ static void draw()
 
                 glWindowPos2iARB(offset * (TILE_SIZE + TILE_SPACE) + 5,
                                  5 + ((offset & 1) * 15));
-                print_string(wrap_modes[i].name);
                 offset++;
             }
         }
-- 
1.7.12.1

_______________________________________________
Piglit mailing list
Piglit@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/piglit

Reply via email to