diff options
| author | Mikhail Gusarov <dottedmag@dottedmag.net> | 2010-06-06 15:28:30 +0700 |
|---|---|---|
| committer | Mikhail Gusarov <dottedmag@dottedmag.net> | 2010-06-06 20:27:18 +0700 |
| commit | 0a4d8cbdcd7b25313fb29ccdb498044af95f7de6 (patch) | |
| tree | 3534c9035180b7e0d2ebbbf0202a370cb61f8e8d /mi/micopy.c | |
| parent | 5a0fc0ad21d562612676ef88ef2d533b2391810a (diff) | |
Remove more superfluous if(p) checks around free(p)
This patch has been generated by the following Coccinelle semantic patch:
@@
expression E;
@@
-if(E) { free(E); }
+free(E);
Signed-off-by: Mikhail Gusarov <dottedmag@dottedmag.net>
Reviewed-by: Julien Cristau <jcristau@debian.org>
Reviewed-by: Fernando Carrijo <fcarrijo@yahoo.com.br>
Reviewed-by: Matt Turner <mattst88@gmail.com>
Diffstat (limited to 'mi/micopy.c')
| -rw-r--r-- | mi/micopy.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/mi/micopy.c b/mi/micopy.c index b1a7a0850..027c461fe 100644 --- a/mi/micopy.c +++ b/mi/micopy.c @@ -108,8 +108,7 @@ miCopyRegion (DrawablePtr pSrcDrawable, pboxNew2 = (BoxPtr)malloc(sizeof(BoxRec) * nbox); if(!pboxNew2) { - if (pboxNew1) - free(pboxNew1); + free(pboxNew1); return; } pboxBase = pboxNext = pbox; @@ -143,10 +142,8 @@ miCopyRegion (DrawablePtr pSrcDrawable, dx, dy, reverse, upsidedown, bitPlane, closure); - if (pboxNew1) - free(pboxNew1); - if (pboxNew2) - free(pboxNew2); + free(pboxNew1); + free(pboxNew2); } RegionPtr |