Browse Source

libdthelp: resolve 46 compiler warnings

Peter Howkins 11 years ago
parent
commit
cd39eabb18

+ 2 - 2
cde/lib/DtHelp/Access.c

@@ -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;
       }
 
     /*

+ 1 - 1
cde/lib/DtHelp/AccessSDL.c

@@ -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;

+ 4 - 4
cde/lib/DtHelp/Callbacks.c

@@ -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;

+ 1 - 1
cde/lib/DtHelp/Destroy.c

@@ -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)

+ 7 - 7
cde/lib/DtHelp/Font.c

@@ -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);

+ 1 - 0
cde/lib/DtHelp/FontAttr.c

@@ -46,6 +46,7 @@
  * system includes
  */
 #include <string.h>
+#include <stdlib.h>
 
 /*
  * private includes

+ 1 - 1
cde/lib/DtHelp/FormatCCDF.c

@@ -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;	*/
   };

+ 1 - 1
cde/lib/DtHelp/GlobSearch.c

@@ -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;
 }
 
 

+ 16 - 15
cde/lib/DtHelp/Graphics.c

@@ -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;

+ 1 - 1
cde/lib/DtHelp/HelpDialog.c

@@ -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 */

+ 1 - 1
cde/lib/DtHelp/HelpQuickD.c

@@ -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,

+ 3 - 3
cde/lib/DtHelp/HelpUtil.c

@@ -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;

+ 1 - 1
cde/lib/DtHelp/LayoutUtil.c

@@ -169,7 +169,7 @@ CheckList (
      */
     if (list != NULL)
       {
-        while (NULL != *list)
+        while ('\0' != *list)
 	  {
 	    /*
 	     * it matches, return true

+ 1 - 1
cde/lib/DtHelp/XInterface.c

@@ -2068,7 +2068,7 @@ _DtHelpDALoadGraphic (
     if (fileName != file_xid)
 	free (fileName);
 
-    if (pGS->pix == NULL)
+    if (pGS->pix == 0)
       {
 	free(pReg);
 	free(pGS);

+ 2 - 2
cde/lib/DtHelp/XUICreate.c

@@ -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;

+ 1 - 1
cde/lib/DtHelp/il/ildecompg3.c

@@ -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;

+ 1 - 1
cde/lib/DtHelp/il/ildecompg4.c

@@ -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;