[PATCH xf86-video-trident] Fix up deprecated/obsolete code.

Tiago Vignatti tiago.vignatti at nokia.com
Mon Aug 2 06:00:41 PDT 2010


On Thu, Jul 29, 2010 at 06:38:57PM +0200, ext Trevor Woerner wrote:
> From: Trevor Woerner <twoerner at gmail.com>
> 
> The compile issues several warnings regarding obsolete or deprecated
> code (moving away from Xalloc(), Xcalloc(), Xfree() etc...). This
> change removes all such warnings.
> 
> Signed-off-by: Trevor Woerner <twoerner at gmail.com>
> ---
>  src/trident_dga.c    |    6 +++---
>  src/trident_driver.c |   18 +++++++++---------
>  src/trident_video.c  |   26 +++++++++++++-------------
>  3 files changed, 25 insertions(+), 25 deletions(-)
> 
> diff --git a/src/trident_dga.c b/src/trident_dga.c
> index 78fa8f4..dd836a6 100644
> --- a/src/trident_dga.c
> +++ b/src/trident_dga.c
> @@ -83,15 +83,15 @@ TRIDENTDGAInit(ScreenPtr pScreen)
>     while(pMode) {
>  
>  	if(0 /*pScrn->displayWidth != pMode->HDisplay*/) {
> -	    newmodes = xrealloc(modes, (num + 2) * sizeof(DGAModeRec));
> +	    newmodes = realloc(modes, (num + 2) * sizeof(DGAModeRec));
>  	    oneMore = TRUE;
>  	} else {
> -	    newmodes = xrealloc(modes, (num + 1) * sizeof(DGAModeRec));
> +	    newmodes = realloc(modes, (num + 1) * sizeof(DGAModeRec));
>  	    oneMore = FALSE;
>  	}
>  
>  	if(!newmodes) {
> -	   xfree(modes);
> +	   free(modes);
>  	   return FALSE;
>  	}
>  	modes = newmodes;
> diff --git a/src/trident_driver.c b/src/trident_driver.c
> index 31cbb83..9192800 100644
> --- a/src/trident_driver.c
> +++ b/src/trident_driver.c
> @@ -63,7 +63,7 @@
>  
>  #ifdef XFreeXDGA
>  #define _XF86DGA_SERVER_
> -#include <X11/extensions/xf86dgastr.h>
> +#include <X11/extensions/xf86dgaproto.h>
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  comment bellow


>  #endif
>  
>  #include "globals.h"
> @@ -560,7 +560,7 @@ TRIDENTFreeRec(ScrnInfoPtr pScrn)
>  {
>      if (pScrn->driverPrivate == NULL)
>  	return;
> -    xfree(pScrn->driverPrivate);
> +    free(pScrn->driverPrivate);
>      pScrn->driverPrivate = NULL;
>  }
>  
> @@ -900,7 +900,7 @@ TRIDENTProbe(DriverPtr drv, int flags)
>  		    foundScreen = TRUE;
>  		}
>  	    }
> -	    xfree(usedChips);
> +	    free(usedChips);
>  	}
>      }
>  
> @@ -937,7 +937,7 @@ TRIDENTProbe(DriverPtr drv, int flags)
>      }
>  #endif    
>  
> -    xfree(devSections);
> +    free(devSections);
>      return foundScreen;
>  }
>  	
> @@ -1163,7 +1163,7 @@ TRIDENTPreInit(ScrnInfoPtr pScrn, int flags)
>      xf86CollectOptions(pScrn, NULL);
>  
>      /* Process the options */
> -    if (!(pTrident->Options = xalloc(sizeof(TRIDENTOptions))))
> +    if (!(pTrident->Options = malloc(sizeof(TRIDENTOptions))))
>  	return FALSE;
>      memcpy(pTrident->Options, TRIDENTOptions, sizeof(TRIDENTOptions));
>      xf86ProcessOptions(pScrn->scrnIndex, pScrn->options, pTrident->Options);
> @@ -2868,7 +2868,7 @@ TRIDENTScreenInit(int scrnIndex, ScreenPtr pScreen, int argc, char **argv)
>  
>      if(pTrident->ShadowFB) {
>   	pTrident->ShadowPitch = BitmapBytePad(pScrn->bitsPerPixel * width);
> -        pTrident->ShadowPtr = xalloc(pTrident->ShadowPitch * height);
> +        pTrident->ShadowPtr = malloc(pTrident->ShadowPitch * height);
>  	displayWidth = pTrident->ShadowPitch / (pScrn->bitsPerPixel >> 3);
>          FBStart = pTrident->ShadowPtr;
>      } else {
> @@ -3238,15 +3238,15 @@ TRIDENTCloseScreen(int scrnIndex, ScreenPtr pScreen)
>  	XAADestroyInfoRec(pTrident->AccelInfoRec);
>      if (pTrident->EXADriverPtr) {
>  	exaDriverFini(pScreen);
> -	xfree(pTrident->EXADriverPtr);
> +	free(pTrident->EXADriverPtr);
>  	pTrident->EXADriverPtr = NULL;
>      }	
>      if (pTrident->CursorInfoRec)
>  	xf86DestroyCursorInfoRec(pTrident->CursorInfoRec);
>      if (pTrident->ShadowPtr)
> -	xfree(pTrident->ShadowPtr);
> +	free(pTrident->ShadowPtr);
>      if (pTrident->DGAModes)
> -	xfree(pTrident->DGAModes);
> +	free(pTrident->DGAModes);
>      pScrn->vtSema = FALSE;
>  
>      if(pTrident->BlockHandler)
> diff --git a/src/trident_video.c b/src/trident_video.c
> index 0c5497b..1ed7db2 100644
> --- a/src/trident_video.c
> +++ b/src/trident_video.c
> @@ -125,7 +125,7 @@ void TRIDENTInitVideo(ScreenPtr pScreen)
>  	    adaptors = &newAdaptor;
>  	} else {
>  	    newAdaptors =  /* need to free this someplace */
> -		xalloc((num_adaptors + 1) * sizeof(XF86VideoAdaptorPtr*));
> +		malloc((num_adaptors + 1) * sizeof(XF86VideoAdaptorPtr*));
>  	    if(newAdaptors) {
>  		memcpy(newAdaptors, adaptors, num_adaptors * 
>  					sizeof(XF86VideoAdaptorPtr));
> @@ -140,7 +140,7 @@ void TRIDENTInitVideo(ScreenPtr pScreen)
>          xf86XVScreenInit(pScreen, adaptors, num_adaptors);
>  
>      if(newAdaptors)
> -	xfree(newAdaptors);
> +	free(newAdaptors);
>  
>      if (pTrident->videoFlags)
>  	xf86DrvMsgVerb(pScrn->scrnIndex,X_INFO,3,
> @@ -321,7 +321,7 @@ TRIDENTSetupImageVideo(ScreenPtr pScreen)
>      XF86VideoAdaptorPtr adapt;
>      TRIDENTPortPrivPtr pPriv;
>  
> -    if(!(adapt = xcalloc(1, sizeof(XF86VideoAdaptorRec) +
> +    if(!(adapt = calloc(1, sizeof(XF86VideoAdaptorRec) +
>  			    sizeof(TRIDENTPortPrivRec) +
>  			    sizeof(DevUnion))))
>  	return NULL;
> @@ -978,18 +978,18 @@ TRIDENTAllocateSurface(
>      surface->width = w;
>      surface->height = h;
>  
> -    if(!(surface->pitches = xalloc(sizeof(int)))) {
> +    if(!(surface->pitches = malloc(sizeof(int)))) {
>  	xf86FreeOffscreenLinear(linear);
>  	return BadAlloc;
>      }
> -    if(!(surface->offsets = xalloc(sizeof(int)))) {
> -	xfree(surface->pitches);
> +    if(!(surface->offsets = malloc(sizeof(int)))) {
> +	free(surface->pitches);
>  	xf86FreeOffscreenLinear(linear);
>  	return BadAlloc;
>      }
> -    if(!(pPriv = xalloc(sizeof(OffscreenPrivRec)))) {
> -	xfree(surface->pitches);
> -	xfree(surface->offsets);
> +    if(!(pPriv = malloc(sizeof(OffscreenPrivRec)))) {
> +	free(surface->pitches);
> +	free(surface->offsets);
>  	xf86FreeOffscreenLinear(linear);
>  	return BadAlloc;
>      }
> @@ -1034,9 +1034,9 @@ TRIDENTFreeSurface(
>      if(pPriv->isOn)
>  	TRIDENTStopSurface(surface);
>      xf86FreeOffscreenLinear(pPriv->linear);
> -    xfree(surface->pitches);
> -    xfree(surface->offsets);
> -    xfree(surface->devPrivate.ptr);
> +    free(surface->pitches);
> +    free(surface->offsets);
> +    free(surface->devPrivate.ptr);
>  
>      return Success;
>  }
> @@ -1128,7 +1128,7 @@ TRIDENTInitOffscreenImages(ScreenPtr pScreen)
>      XF86OffscreenImagePtr offscreenImages;
>  
>      /* need to free this someplace */
> -    if(!(offscreenImages = xalloc(sizeof(XF86OffscreenImageRec))))
> +    if(!(offscreenImages = malloc(sizeof(XF86OffscreenImageRec))))
>  	return;
>  
>      offscreenImages[0].image = &Images[0];
> -- 
> 1.7.1

looks nice but next time would be better if you split in another patch those
lines that you fixed prototype declaration:

Reviewed-by: Tiago Vignatti <tiago.vignatti at nokia.com>

Do you have commit rights to apply?

             Tiago


More information about the xorg-devel mailing list