Changeset 33985 for branches/eam_branches/ipp-20120601/Ohana/src/kapa2/src/CheckVisual.c

Timestamp:

Jun 6, 2012, 9:16:29 PM (14 years ago)

Author:

eugene

Message:

gcc 4.6.3 adds a new level of pedantry: if a variable is defined and set, but not used, it raises a warning. with -Werror, this forces a lot of minor fixes. I do not think any of the resultings changes caught any real problems, at least not with any commonly used code

File:

• branches/eam_branches/ipp-20120601/Ohana/src/kapa2/src/CheckVisual.c (modified) (4 diffs)

branches/eam_branches/ipp-20120601/Ohana/src/kapa2/src/CheckVisual.c

@@ -10 +10 @@
 
   int i, Nfound, N;
-  int isColor, isDefault;
+  int isDefault;
   XVisualInfo *visual_list, visual_temp;
   unsigned long planes[3];
@@ -35 +35 @@
   
   // set these based on selected visual
-  isColor = isDefault = FALSE;
+  isDefault = FALSE;
 
   // attempt to select the most desirable type of visual: TrueColor
   for (i = 0; (i < Nfound) && (visual_list[i].class != TrueColor); i++);
   if (i != Nfound) {
-    isColor = TRUE;
+    // isColor = TRUE;
     if (DEBUG) fprintf (stderr, "visual class is %d\n", visual_list[i].class);
     isDefault = (graphic[0].visual == visual_list[i].visual);
@@ -52 +52 @@
   for (i = 0; (i < Nfound) && (visual_list[i].class != DirectColor); i++);
   if (i != Nfound) {
-    isColor = TRUE;
+    // isColor = TRUE;
     if (DEBUG) fprintf (stderr, "visual class is %d\n", visual_list[i].class);
     isDefault = (graphic[0].visual == visual_list[i].visual);
@@ -64 +64 @@
   for (i = 0; (i < Nfound) && (visual_list[i].class != PseudoColor); i++);
   if (i != Nfound) {
-    isColor = TRUE;
+    // isColor = TRUE;
     if (DEBUG) fprintf (stderr, "selected visual class is %d\n", visual_list[i].class);
     isDefault = (graphic[0].visual == visual_list[i].visual);