[PATCH] exa: don't need to check for NULL pointer if we already assumed it has a value

Tiago Vignatti tiago.vignatti at nokia.com
Fri Apr 16 08:42:24 PDT 2010


the alternative would be to check ps in the beginning of the function.

Signed-off-by: Tiago Vignatti <tiago.vignatti at nokia.com>
---
 exa/exa.c |    2 --
 1 files changed, 0 insertions(+), 2 deletions(-)

diff --git a/exa/exa.c b/exa/exa.c
index 590d9a5..51697bc 100644
--- a/exa/exa.c
+++ b/exa/exa.c
@@ -779,14 +779,12 @@ exaCloseScreen(int i, ScreenPtr pScreen)
     unwrap(pExaScr, pScreen, BitmapToRegion);
     unwrap(pExaScr, pScreen, CreateScreenResources);
 #ifdef RENDER
-    if (ps) {
 	unwrap(pExaScr, ps, Composite);
 	if (pExaScr->SavedGlyphs)
 	    unwrap(pExaScr, ps, Glyphs);
 	unwrap(pExaScr, ps, Trapezoids);
 	unwrap(pExaScr, ps, Triangles);
 	unwrap(pExaScr, ps, AddTraps);
-    }
 #endif
 
     xfree (pExaScr);
-- 
1.6.0.4



More information about the xorg-devel mailing list