diff options
author | Tiago Vignatti <tiago.vignatti@nokia.com> | 2010-04-19 20:34:30 +0300 |
---|---|---|
committer | Tiago Vignatti <tiago.vignatti@nokia.com> | 2010-04-21 18:07:22 +0300 |
commit | 471f350c99b2e7c5a9045eef5ebff65cd2ddb7c5 (patch) | |
tree | c6db1ad0b82d32e5f1c2b17d79142b2e2e9e626f /exa | |
parent | 96784f4fcb6c2fb82f6d1abbd28ea1e189e4e6f5 (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>
Diffstat (limited to 'exa')
-rw-r--r-- | exa/exa.c | 14 |
1 files changed, 6 insertions, 8 deletions
@@ -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); |