summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTiago Vignatti <tiago.vignatti@nokia.com>2010-04-19 20:34:30 +0300
committerTiago Vignatti <tiago.vignatti@nokia.com>2010-04-21 18:07:22 +0300
commit471f350c99b2e7c5a9045eef5ebff65cd2ddb7c5 (patch)
treec6db1ad0b82d32e5f1c2b17d79142b2e2e9e626f
parent96784f4fcb6c2fb82f6d1abbd28ea1e189e4e6f5 (diff)
exa: don't need to check for NULL pointer if we already assumed it has a value
the alternative would be to check ps in the beginning of the function. Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com> Acked-by: Michel Dänzer <michel@daenzer.net>
-rw-r--r--exa/exa.c14
1 files changed, 6 insertions, 8 deletions
diff --git a/exa/exa.c b/exa/exa.c
index 590d9a500..b2bd1c565 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);
- }
+ 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);