From ef1a771fde77e29b8bf327905674da59ea3d0bf0 Mon Sep 17 00:00:00 2001
From: Robert Bragg <robert@linux.intel.com>
Date: Wed, 27 May 2009 23:39:18 +0100
Subject: [PATCH] [build] Fixes some compiler warnings when building for GLES 2

There were a number of variables shadowing other symbols, and an unused
display variable.
---
 clutter/cogl/gles/cogl-gles2-wrapper.c | 20 +++++++-------
 clutter/cogl/gles/cogl-primitives.c    | 36 +++++++++++++-------------
 tests/conform/test-conform-main.c      |  3 +--
 3 files changed, 29 insertions(+), 30 deletions(-)

diff --git a/clutter/cogl/gles/cogl-gles2-wrapper.c b/clutter/cogl/gles/cogl-gles2-wrapper.c
index 523676dd2..aec15b7a0 100644
--- a/clutter/cogl/gles/cogl-gles2-wrapper.c
+++ b/clutter/cogl/gles/cogl-gles2-wrapper.c
@@ -89,13 +89,13 @@ cogl_gles2_wrapper_create_shader (GLenum type, const char *source)
 
   if (!status)
     {
-      char log[1024];
+      char shader_log[1024];
       GLint len;
 
-      glGetShaderInfoLog (shader, sizeof (log) - 1, &len, log);
-      log[len] = '\0';
+      glGetShaderInfoLog (shader, sizeof (shader_log) - 1, &len, shader_log);
+      shader_log[len] = '\0';
 
-      g_critical ("%s", log);
+      g_critical ("%s", shader_log);
 
       glDeleteShader (shader);
 
@@ -538,9 +538,9 @@ cogl_gles2_wrapper_get_program (const CoglGles2WrapperSettings *settings)
 	  CoglShader *shader
 	    = _cogl_shader_pointer_from_handle ((CoglHandle) node->data);
 
-	  if (shader->type == CGL_VERTEX_SHADER)
+	  if (shader->type == COGL_SHADER_TYPE_VERTEX)
 	    custom_vertex_shader = TRUE;
-	  else if (shader->type == CGL_FRAGMENT_SHADER)
+	  else if (shader->type == COGL_SHADER_TYPE_FRAGMENT)
 	    custom_fragment_shader = TRUE;
 	}
     }
@@ -581,13 +581,13 @@ cogl_gles2_wrapper_get_program (const CoglGles2WrapperSettings *settings)
 
   if (!status)
     {
-      char log[1024];
+      char shader_log[1024];
       GLint len;
 
-      glGetProgramInfoLog (program->program, sizeof (log) - 1, &len, log);
-      log[len] = '\0';
+      glGetProgramInfoLog (program->program, sizeof (shader_log) - 1, &len, shader_log);
+      shader_log[len] = '\0';
 
-      g_critical ("%s", log);
+      g_critical ("%s", shader_log);
 
       glDeleteProgram (program->program);
       g_slice_free (CoglGles2WrapperProgram, program);
diff --git a/clutter/cogl/gles/cogl-primitives.c b/clutter/cogl/gles/cogl-primitives.c
index 1e03e1ea2..24b6b0759 100644
--- a/clutter/cogl/gles/cogl-primitives.c
+++ b/clutter/cogl/gles/cogl-primitives.c
@@ -361,30 +361,30 @@ _cogl_path_fill_nodes_scanlines (CoglPathNode *path,
         while (iter)
           {
             GSList *next = iter->next;
-            GLfloat x0, x1;
-            GLfloat y0, y1;
+            GLfloat x_0, x_1;
+            GLfloat y_0, y_1;
             if (!next)
               break;
 
-            x0 = GPOINTER_TO_INT (iter->data);
-            x1 = GPOINTER_TO_INT (next->data);
-            y0 = bounds_y + i;
-            y1 = bounds_y + i + 1.0625f;
+            x_0 = GPOINTER_TO_INT (iter->data);
+            x_1 = GPOINTER_TO_INT (next->data);
+            y_0 = bounds_y + i;
+            y_1 = bounds_y + i + 1.0625f;
             /* render scanlines 1.0625 high to avoid gaps when
                transformed */
 
-            coords[span_no * 12 + 0] = x0;
-            coords[span_no * 12 + 1] = y0;
-            coords[span_no * 12 + 2] = x1;
-            coords[span_no * 12 + 3] = y0;
-            coords[span_no * 12 + 4] = x1;
-            coords[span_no * 12 + 5] = y1;
-            coords[span_no * 12 + 6] = x0;
-            coords[span_no * 12 + 7] = y0;
-            coords[span_no * 12 + 8] = x0;
-            coords[span_no * 12 + 9] = y1;
-            coords[span_no * 12 + 10] = x1;
-            coords[span_no * 12 + 11] = y1;
+            coords[span_no * 12 + 0] = x_0;
+            coords[span_no * 12 + 1] = y_0;
+            coords[span_no * 12 + 2] = x_1;
+            coords[span_no * 12 + 3] = y_0;
+            coords[span_no * 12 + 4] = x_1;
+            coords[span_no * 12 + 5] = y_1;
+            coords[span_no * 12 + 6] = x_0;
+            coords[span_no * 12 + 7] = y_0;
+            coords[span_no * 12 + 8] = x_0;
+            coords[span_no * 12 + 9] = y_1;
+            coords[span_no * 12 + 10] = x_1;
+            coords[span_no * 12 + 11] = y_1;
             span_no ++;
             iter = next->next;
           }
diff --git a/tests/conform/test-conform-main.c b/tests/conform/test-conform-main.c
index b31ab524b..46ccef07a 100644
--- a/tests/conform/test-conform-main.c
+++ b/tests/conform/test-conform-main.c
@@ -60,11 +60,10 @@ int
 main (int argc, char **argv)
 {
   TestConformSharedState *shared_state = g_new0 (TestConformSharedState, 1);
-  const gchar *display;
 
 #ifdef HAVE_CLUTTER_GLX
   /* on X11 we need a display connection to run the test suite */
-  display = g_getenv ("DISPLAY");
+  const gchar *display = g_getenv ("DISPLAY");
   if (!display || *display == '\0')
     {
       g_print ("No DISPLAY found. Unable to run the conformance "