diff options
| author | Paulo Cesar Pereira de Andrade <pcpa@mandriva.com.br> | 2008-11-24 15:15:43 -0200 |
|---|---|---|
| committer | Paulo Cesar Pereira de Andrade <pcpa@mandriva.com.br> | 2008-11-24 15:15:43 -0200 |
| commit | 26ce0b163d9cf44ec1bc1d1bd460887aa5b405a7 (patch) | |
| tree | 5063cb22f76ece5925f2bbfa6713f978715b770a /src/smilynx_crtc.c | |
| parent | 2057d7ae05ef269f75a4d53fe5c4914a10856cb1 (diff) | |
Use a single debug macro for leaving functions.
The LEAVE() macro, when debug is disabled, now is defined as:
<hash>define LEAVE(...) return __VA_ARGS__
This avoids the requirement of a return statement after the LEAVE()
macro call, what can be confusing.
Diffstat (limited to 'src/smilynx_crtc.c')
| -rw-r--r-- | src/smilynx_crtc.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/smilynx_crtc.c b/src/smilynx_crtc.c index 4dd5719..f56fd2e 100644 --- a/src/smilynx_crtc.c +++ b/src/smilynx_crtc.c @@ -250,7 +250,6 @@ SMILynx_CrtcModeSet_vga(xf86CrtcPtr crtc, if (!vgaHWInit(pScrn, mode)) { LEAVE(); - return; } if ((mode->HDisplay == 640) && SMI_LYNXM_SERIES(pSmi->Chipset)) { @@ -710,7 +709,7 @@ SMILynx_CrtcPreInit(ScrnInfoPtr pScrn) crtc0=xf86CrtcCreate(pScrn,&SMILynx_Crtc0Funcs); if(!crtc0) - RETURN(FALSE); + LEAVE(FALSE); crtc0->driver_private = &SMILynx_Crtc0Priv; }else{ if(pSmi->Dualhead){ @@ -723,7 +722,7 @@ SMILynx_CrtcPreInit(ScrnInfoPtr pScrn) crtc0=xf86CrtcCreate(pScrn,&SMILynx_Crtc0Funcs); if(!crtc0) - RETURN(FALSE); + LEAVE(FALSE); crtc0->driver_private = &SMILynx_Crtc0Priv; /* CRTC1 is CRT */ @@ -735,7 +734,7 @@ SMILynx_CrtcPreInit(ScrnInfoPtr pScrn) crtc1=xf86CrtcCreate(pScrn,&SMILynx_Crtc1Funcs); if(!crtc1) - RETURN(FALSE); + LEAVE(FALSE); crtc1->driver_private = &SMILynx_Crtc1Priv; }else{ @@ -757,11 +756,11 @@ SMILynx_CrtcPreInit(ScrnInfoPtr pScrn) crtc0=xf86CrtcCreate(pScrn,&SMILynx_Crtc0Funcs); if(!crtc0) - RETURN(FALSE); + LEAVE(FALSE); crtc0->driver_private = &SMILynx_Crtc0Priv; } } - RETURN(TRUE); + LEAVE(TRUE); } |