libdthelp: resolve 46 compiler warnings
authorPeter Howkins <flibble@users.sf.net>
Mon, 8 Oct 2012 14:34:45 +0000 (15:34 +0100)
committerPeter Howkins <flibble@users.sf.net>
Mon, 8 Oct 2012 14:34:45 +0000 (15:34 +0100)
17 files changed:
cde/lib/DtHelp/Access.c
cde/lib/DtHelp/AccessSDL.c
cde/lib/DtHelp/Callbacks.c
cde/lib/DtHelp/Destroy.c
cde/lib/DtHelp/Font.c
cde/lib/DtHelp/FontAttr.c
cde/lib/DtHelp/FormatCCDF.c
cde/lib/DtHelp/GlobSearch.c
cde/lib/DtHelp/Graphics.c
cde/lib/DtHelp/HelpDialog.c
cde/lib/DtHelp/HelpQuickD.c
cde/lib/DtHelp/HelpUtil.c
cde/lib/DtHelp/LayoutUtil.c
cde/lib/DtHelp/XInterface.c
cde/lib/DtHelp/XUICreate.c
cde/lib/DtHelp/il/ildecompg3.c
cde/lib/DtHelp/il/ildecompg4.c

index eec7fe5524259795199b2d459110a7bac403b570..a11dc8b902388447b921625db53a876521a5bdcc 100644 (file)
@@ -696,7 +696,7 @@ _DtHelpCeExpandPathname (
 #define        MY_NUM 7
     _DtSubstitutionRec mySubs [MY_NUM];
 
-    if (spec == NULL || *spec == NULL)
+    if (spec == NULL || *spec == '\0')
       {
        errno = EINVAL;
        return NULL;
@@ -2404,7 +2404,7 @@ _DtHelpCeGetDocStamp (
       {
        errno = EINVAL;
        _DtHelpProcessUnlock();
-       return NULL;
+       return 0;
       }
 
     /*
index fb7b865572adaeb751a2c2dd46bbc73766ed7958..aaffae8ea20c976dfcb0fc90326ecfdc558ff09f 100644 (file)
@@ -1127,7 +1127,7 @@ _DtHelpCeGetSdlIdPath(
     *ret_ids = NULL;
 
     if (_DtHelpCeGetSdlVolIds(volume, -1, &idSegs) != 0)
-       return NULL;
+       return 0;
 
     if (_SdlVolumeMinorNumber(_DtHelpCeGetSdlVolumePtr(volume)) >= SDL_DTD_1_1)
        hiddenNo = 0;
index ce025a1dc58188c476017904d465987b025846b5..10b9278343cbae15069b4cd44e962151ece6d786 100644 (file)
@@ -351,7 +351,7 @@ ScrollTimerCB (
     if (*id != pDAS->scr_timer_id)
       return;
 
-    pDAS->scr_timer_id = NULL;
+    pDAS->scr_timer_id = 0;
 
     maxY  = pDAS->maxYpos;
     dispY = pDAS->firstVisible + pDAS->dispUseHeight;
@@ -604,7 +604,7 @@ _DtHelpCancelSelection(
        if (pDAS->scr_timer_id)
          {
            XtRemoveTimeOut (pDAS->scr_timer_id);
-           pDAS->scr_timer_id = NULL;
+           pDAS->scr_timer_id = 0;
          }
        _DtHelpClearSelection (client_data);
       }
@@ -1067,7 +1067,7 @@ _DtHelpEndSelectionCB (
     if (pDAS->scr_timer_id)
       {
        XtRemoveTimeOut (pDAS->scr_timer_id);
-       pDAS->scr_timer_id = NULL;
+       pDAS->scr_timer_id = 0;
       }
 
     newX = callback->event->xbutton.x;
@@ -1140,7 +1140,7 @@ _DtHelpMouseMoveCB(
     if (pDAS->scr_timer_id)
       {
        XtRemoveTimeOut (pDAS->scr_timer_id);
-       pDAS->scr_timer_id = NULL;
+       pDAS->scr_timer_id = 0;
       }
 
     newX = event->xmotion.x;
index 982288701017c11f8e33fb2c76ccd0af471e91e0..ac92912ce4cbdd04bb2ffb7b4c4babf232dfcfbb 100644 (file)
@@ -161,7 +161,7 @@ _DtHelpDisplayAreaDestroyCB (
     XFreeGC (dpy, pDAS->normalGC);
     XFreeGC (dpy, pDAS->pixmapGC);
     XFreeGC (dpy, pDAS->invertGC);
-    if (pDAS->def_pix != NULL &&
+    if (pDAS->def_pix != 0 &&
        XmDestroyPixmap(XDefaultScreenOfDisplay(dpy), pDAS->def_pix) == False)
            XFreePixmap(dpy, pDAS->def_pix);
     if (pDAS->context != NULL)
index 20d05521bfc61dcfe4afeb0a3903d3cabc842b1e..d34d96c23ba6499265e5a4eb5e87319c467e6e9b 100644 (file)
@@ -264,7 +264,7 @@ static  const char    *FontResources[] =
 #define        GROW_SIZE       5
 
 static Boolean         QuarksInited = FALSE;
-static XrmQuark        StringFontQuark = NULL;
+static XrmQuark        StringFontQuark = 0;
 static XrmName         DefaultFontQuarks[_DtHelpFontQuarkNumber];
 static XrmBinding      FontBindings[_DtHelpFontQuarkNumber] =
         { XrmBindLoosely, XrmBindLoosely, XrmBindLoosely, XrmBindLoosely,
@@ -272,7 +272,7 @@ static      XrmBinding      FontBindings[_DtHelpFontQuarkNumber] =
 
 static DtHelpDAFSMetrics  DefaultMetrics = { FALSE, 0, 0 };
 static DtHelpDAFontInfo   DefFontInfo    = { NULL, NULL, NULL, NULL, NULL,
-                                       NULL, NULL, NULL, 0, 0, 0, 0, 0};
+                                       NULL, 0, NULL, 0, 0, 0, 0, 0};
 
 /******************************************************************************
  *
@@ -546,7 +546,7 @@ __DtHelpFontIndexGet (
        retValue.addr = (XtPointer) &xrm_list[_DT_HELP_FONT_CHAR_SET];
        xrmList[0] = XrmStringToQuark (buffer);
        xrmList[1] = XrmStringToQuark ("code_set");
-       xrmList[2] = NULL;
+       xrmList[2] = 0;
        XrmQPutResource (&(fontInfo->font_idx_db),
                                ((XrmBindingList) FontBindings),
                                        xrmList, _DtHelpXrmQuark, &retValue);
@@ -596,7 +596,7 @@ __DtHelpFontCharSetQuarkGet (
     sprintf (buffer, "%d", font_index);
     xrmList[0] = XrmStringToQuark (buffer);
     xrmList[1] = XrmStringToQuark ("code_set");
-    xrmList[2] = NULL;
+    xrmList[2] = 0;
 
     /*
      * look in my font data base for the quark.
@@ -640,7 +640,7 @@ __DtHelpFontLangQuarkGet (
     sprintf (buffer, "%d", font_index);
     xrmList[0] = XrmStringToQuark (buffer);
     xrmList[1] = XrmStringToQuark ("language");
-    xrmList[2] = NULL;
+    xrmList[2] = 0;
 
     /*
      * look in my font data base for the quark.
@@ -743,7 +743,7 @@ __DtHelpFontDatabaseInit (
         DefaultFontQuarks[_DT_HELP_FONT_LANG_TER] = XrmStringToQuark ("C");
         DefaultFontQuarks[_DT_HELP_FONT_CHAR_SET] =
                                                XrmStringToQuark ("ISO-8859-1");
-        DefaultFontQuarks[_DT_HELP_FONT_END]      = NULL;
+        DefaultFontQuarks[_DT_HELP_FONT_END]      = 0;
        QuarksInited = True;
       }
     _DtHelpProcessUnlock();
@@ -1053,7 +1053,7 @@ _DtHelpGetExactFontIndex (
                    retValue.addr = (XtPointer) &myQuark;
                    xrmList[0] = XrmStringToQuark (buffer);
                    xrmList[1] = XrmStringToQuark ("code_set");
-                   xrmList[2] = NULL;
+                   xrmList[2] = 0;
                    XrmQPutResource (&(fontInfo->font_idx_db),
                                    ((XrmBindingList) FontBindings),
                                    xrmList, _DtHelpXrmQuark, &retValue);
index 25541516eafbcc8c80ada62954e32787f3531928..e5065950b00d75030a63dd48854b891dc020e93c 100644 (file)
@@ -46,6 +46,7 @@
  * system includes
  */
 #include <string.h>
+#include <stdlib.h>
 
 /*
  * private includes
index c8ce2511109a5ab0fa4cb2fdf62db7dc5d4383d5..76ee8c7a36e6522152c0d45917010a9b929a7634 100644 (file)
@@ -390,7 +390,7 @@ static const FrmtPrivateInfo DefPrivInfo =
        (char) False,           /* char          top_block;     */
        (char) False,           /* char          dup_flag;      */
        0,                      /* int           sdl_type;      */
-       NULL,                   /* void         *match_info;    */
+       0,                      /* void         *match_info;    */
        NULL,                   /* void         *id_info;       */
        NULL,                   /* void         *tmp_info;      */
   };
index 48103e57db56be1c902f6eb87ff6571532716b42..d4d0914c1d8aaa791603c497a592568ea032e0b8 100644 (file)
@@ -5195,7 +5195,7 @@ void  _DtHelpGlobSrchInitVars(
     srch->curVolRadBtnSens = False;
     srch->volLeftCnt    = 0;
     srch->volListHead   = NULL;        /* info on search topics found */
-    srch->workProcId    = NULL;
+    srch->workProcId    = 0;
 }
 
 \f
index 7631c7669c012e634456c435558c4f26ab96af3f..6173df221acb71e8e013c63c7559dd759d035fd1 100644 (file)
@@ -1311,7 +1311,7 @@ static enum _DtGrLoadStatus processBitmap(
     int           junk;
     unsigned int  width, height;
     unsigned char *data;
-    Pixmap        scaled_pixmap=NULL;
+    Pixmap        scaled_pixmap = 0;
     Display       *dpy = DisplayOfScreen(screen);
     Drawable      drawable = RootWindowOfScreen(screen);
     XImage        ximage;
@@ -1404,7 +1404,7 @@ static enum _DtGrLoadStatus processTiff(
 )
 {
     int                 result = -1;
-    Pixmap       pixmap = NULL;
+    Pixmap       pixmap = 0;
     ilFile      inFile;
     ilPipe      inPipe;
     ilFileImage         inImage;
@@ -1507,7 +1507,7 @@ static enum _DtGrLoadStatus processTiff(
                if (result == -1)
                  {
                    XFreePixmap (dpy, pixmap);
-                   pixmap = NULL;
+                   pixmap = 0;
                  }
              }
          }
@@ -1650,7 +1650,8 @@ myXpmReadFileToPixmap(
     float           ratio
 )
 {
-    XImage *image, **imageptr = NULL;
+    XImage *image = NULL;
+    XImage **imageptr = 0;
     XImage *shapeimage, **shapeimageptr = NULL;
     int ErrorStatus;
     int switchFlag = 0;
@@ -1660,11 +1661,11 @@ myXpmReadFileToPixmap(
      */
 
     if (pixmap_return) {
-        *pixmap_return = NULL;
+        *pixmap_return = 0;
         imageptr = &image;
     }
     if (shapemask_return) {
-        *shapemask_return = NULL;
+        *shapemask_return = 0;
         shapeimageptr = &shapeimage;
     }
 
@@ -1795,7 +1796,7 @@ static enum _DtGrLoadStatus processXpm(
     int                    i, j;
     int             result;
     short           done;
-    Pixmap          pixmap  = NULL;
+    Pixmap          pixmap  = 0;
     XpmAttributes   xpmAttr;
     Visual          vis2;
     XGCValues       gcvalues;
@@ -1990,7 +1991,7 @@ static enum _DtGrLoadStatus processGIF(
                                      g.f_black, g.f_white, ratio);
 
     /* Set the returned colors parameters */
-    if (*ret_pixmap != NULL)
+    if (*ret_pixmap != 0)
     {
         if (g.f_do_visual == DO_COLOR)
        {
@@ -2384,7 +2385,7 @@ _DtHelpProcessGraphic(
     unsigned int    pixHeight = 0; 
     Dimension       pWidth, pHeight;
     char  *ptr;
-    Pixmap pix = NULL, mask = NULL;
+    Pixmap pix = 0, mask = 0;
     _DtGrStream stream;
     Screen *scrptr = ScreenOfDisplay (dpy, screen);
     static enum _DtGrColorModel ForceColor = _DtGrCOLOR;
@@ -2465,7 +2466,7 @@ _DtHelpProcessGraphic(
        result = _DtGrOpenFile(&stream, filename);
 
        if (result != 0)
-           pix = NULL;
+           pix = 0;
         else 
        {
            /* Create a pixmap from the image data stream */
@@ -2481,7 +2482,7 @@ _DtHelpProcessGraphic(
         }
       }
 
-    if (pix == NULL)
+    if (pix == 0)
       {
        /*
         * Try to get a localized pixmap
@@ -2489,7 +2490,7 @@ _DtHelpProcessGraphic(
          * This is a cached pixmap.....
          * Make sure XFreePixmaps does not free this one.
         */
-       if (*def_pix == NULL)
+       if (*def_pix == 0)
          {
            *def_pix = XmGetPixmap(scrptr, "Dthgraphic", fore_ground, back_ground);
            if (*def_pix != XmUNSPECIFIED_PIXMAP)
@@ -2503,16 +2504,16 @@ _DtHelpProcessGraphic(
                                &pixWidth, &pixHeight, &border, &depth) == 0)
                  {
                    XmDestroyPixmap(scrptr, *def_pix);
-                   *def_pix = NULL;
+                   *def_pix = 0;
                  }
              }
            else
-               *def_pix = NULL;
+               *def_pix = 0;
 
            /*
             * couldn't get a localized pixmap, go with a build in default
             */
-           if (*def_pix == NULL)
+           if (*def_pix == 0)
              {
                pixWidth  = Missing_bm_width;
                pixHeight = Missing_bm_height;
index 47fffd6050eb79bb26ca2c5610ef59faf576afd1..9ba39e9ad6401aec91a62e29a1bbbb8f39031516 100644 (file)
@@ -3089,7 +3089,7 @@ static void CloseHelpCB (
        callDataInfo.locationId = NULL;
        callDataInfo.helpVolume = NULL;
        callDataInfo.specification = NULL;
-       callDataInfo.hyperType = NULL;
+       callDataInfo.hyperType = 0;
    
        
        /* All we do is envoke the applications close callback */
index b2a1b6f79afcf722176b3e452aab9c6e329d3a40..fdb41cd871f887cc64e69f5491000c6e49e0eefc 100644 (file)
@@ -2535,7 +2535,7 @@ static void CloseQuickCB (
        callDataInfo.locationId = NULL;
        callDataInfo.helpVolume = NULL;
        callDataInfo.specification = NULL;
-       callDataInfo.hyperType = NULL;
+       callDataInfo.hyperType = 0;
           
        /* All we do is envoke the applications close callback */
        XtCallCallbackList((Widget)qw,qw->qhelp_dialog.qhelp.closeCallback,
index 38967729e62c5bdca34d34a7ad689382c8a7edd0..b5c855e9889ed70673b8caa2c09f4d81e0fb489e 100644 (file)
@@ -1160,7 +1160,7 @@ static Widget  LocateWidgetId(
     Widget           shellWidget,
     Cursor           cursorIn)
 {
-    static Cursor    DfltOnItemCursor = NULL;
+    static Cursor    DfltOnItemCursor = 0;
     Widget           widget;
     Widget           child;
     CompositeWidget  comp_widget;
@@ -1179,7 +1179,7 @@ static Widget  LocateWidgetId(
 
  
     /* Make the target cursor */
-    if (cursorIn != NULL)
+    if (cursorIn != 0)
         cursor = cursorIn;
     else
 #if 0
@@ -1187,7 +1187,7 @@ static Widget  LocateWidgetId(
 #else
     {
         _DtHelpProcessLock();
-        if (NULL == DfltOnItemCursor)
+        if (0 == DfltOnItemCursor)
         {
             char        *bits;
             char        *maskBits;
index 55dee833619331989cfec112b130c385f57e8ab4..57f677bb3b397a476ab85a8775a7b6fcf106196e 100644 (file)
@@ -169,7 +169,7 @@ CheckList (
      */
     if (list != NULL)
       {
-        while (NULL != *list)
+        while ('\0' != *list)
          {
            /*
             * it matches, return true
index 4983393031aba8bd98813d0a9a44c52833434b13..f230715187184f40671acc9280293cbfb300dcc3 100644 (file)
@@ -2068,7 +2068,7 @@ _DtHelpDALoadGraphic (
     if (fileName != file_xid)
        free (fileName);
 
-    if (pGS->pix == NULL)
+    if (pGS->pix == 0)
       {
        free(pReg);
        free(pGS);
index c417fbd06f6ed6551f14909a5f874df08d7d86db..368b6d514df01ba0550fc5c7608f86a4dce412f6 100644 (file)
@@ -441,8 +441,8 @@ HelpCreateDA(
     pDAS->searchColor   = DA_args.search_color;
     pDAS->depth                = 0;
     pDAS->spc_chars     = NULL;
-    pDAS->scr_timer_id  = NULL;
-    pDAS->def_pix       = NULL;
+    pDAS->scr_timer_id  = 0;
+    pDAS->def_pix       = 0;
     pDAS->context       = NULL;
     pDAS->vertIsMapped  = False;
     pDAS->horzIsMapped  = False;
index 8ab2b5b211bf89141865bbbad7313e9897f7baa5..5e08d194a6ea26bde4c53f0168c7c08bdd3d0b2f 100644 (file)
@@ -454,7 +454,7 @@ ilImageDes         *pimdes
        dstdata.producerObject  =  (ilObject) NULL;
        des                     = *pimdes;
        des.compression         =  IL_UNCOMPRESSED;
-       des.compInfo.g4.flags   =  NULL;
+       des.compInfo.g4.flags   =  0;
        dstdata.pDes            =  &des;
        dstdata.pFormat         =  IL_FORMAT_BIT;
        dstdata.width           =  pinfo->width;
index 8876de8d470a69af8648f20047a9e71795276956..747b777045c0dc0c00cc5f2f4f3ebc025289b900 100644 (file)
@@ -615,7 +615,7 @@ ilImageDes         *pimdes
        dstdata.producerObject  =  (ilObject) NULL;
        des                     = *pimdes;
        des.compression         =  IL_UNCOMPRESSED;
-       des.compInfo.g4.flags   =  NULL;
+       des.compInfo.g4.flags   =  0;
        dstdata.pDes            =  &des;
        dstdata.pFormat         =  IL_FORMAT_BIT;
        dstdata.width           =  pinfo->width;