Cleanups for DebugUtil

[Steve Olson]

OK, this one isn't so critical.  The following patch
fixes compiler warnings abount unreachable code and runtime warnings
from Purify about using free when XtFree should be used.

And don't forget the GC leaks in TextF.  You may want to modify
my previous patch to check for freeing NULL GCs.

- Steve

--- lib/Xm/DebugUtil.c.orig     Mon May  1 07:47:31 2000
+++ lib/Xm/DebugUtil.c  Fri Aug  4 12:27:09 2000
@@ -206,16 +206,15 @@
     {
     case XmDROP_DOWN_LIST:
        return("XmDROP_DOWN_LIST");
-       break;
+
     case XmDROP_DOWN_COMBO_BOX:
        return("XmDROP_DOWN_COMBO_BOX");
-       break;
+
     case XmCOMBO_BOX:
        return("XmCOMBO_BOX");
-       break;
+
     default:
        return("UNKNOWN");
-       break;
     }
 }
 #endif
@@ -227,19 +226,18 @@
        {
        case XmGET_ACTUAL_SIZE:
                return("XmGET_ACTUAL_SIZE");
-               break;
+
        case XmGET_PREFERRED_SIZE:
                return("XmGET_PREFERRED_SIZE");
-               break;
+
        case XmGEO_PRE_SET:
                return("XmGEO_PRE_SET");
-               break;
+
        case XmGEO_POST_SET:
                return("XmGEO_POST_SET");
-               break;
+
        default:
                return("Unknown geo action");
-               break;
        }
 }

@@ -632,16 +630,16 @@
     {
     case XmMENU_FOCUS_SAVE:
        return ("XmMENU_FOCUS_SAVE");
-       break;
+
     case XmMENU_FOCUS_RESTORE:
        return ("XmMENU_FOCUS_RESTORE");
-       break;
+
     case XmMENU_FOCUS_SET:
        return ("XmMENU_FOCUS_SET");
-       break;
+
     default:
        return("Unknown focus op");
-       break;
+
     }
 }

@@ -726,11 +724,11 @@
 char *
 XdbXmString2String(XmString xms)
 {
-  static char *s = NULL;
+    static char *s = NULL;

     if (s) {
-      free(s);
-      s=NULL;
+        XtFree(s);
+        s = NULL;
     }
     if (xms == (XmString)XmUNSPECIFIED)
     {



--== Sent via Deja.com http://www.deja.com/ ==--
Before you buy.