Michael Stefaniuc : gdiplus: Remove redundant NULL check before GdipFree ( Smatch).

Alexandre Julliard julliard at winehq.org
Thu Jul 3 06:35:17 CDT 2008


Module: wine
Branch: master
Commit: 808c5dbece015fd9183327a78e120d7b70811b29
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=808c5dbece015fd9183327a78e120d7b70811b29

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Wed Jul  2 23:39:01 2008 +0200

gdiplus: Remove redundant NULL check before GdipFree (Smatch).

---

 dlls/gdiplus/font.c       |    2 +-
 dlls/gdiplus/tests/font.c |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/gdiplus/font.c b/dlls/gdiplus/font.c
index 98a4c43..78de0a6 100644
--- a/dlls/gdiplus/font.c
+++ b/dlls/gdiplus/font.c
@@ -445,7 +445,7 @@ GpStatus WINGDIPAPI GdipDeleteFontFamily(GpFontFamily *FontFamily)
         return InvalidParameter;
     TRACE("Deleting %p (%s)\n", FontFamily, debugstr_w(FontFamily->FamilyName));
 
-    if (FontFamily->FamilyName) GdipFree (FontFamily->FamilyName);
+    GdipFree (FontFamily->FamilyName);
     GdipFree (FontFamily);
 
     return Ok;
diff --git a/dlls/gdiplus/tests/font.c b/dlls/gdiplus/tests/font.c
index 93efbc4..59a8d53 100644
--- a/dlls/gdiplus/tests/font.c
+++ b/dlls/gdiplus/tests/font.c
@@ -190,7 +190,7 @@ static void test_fontfamily (void)
     stat = GdipDeleteFontFamily(*family);
     expect (Ok, stat);
 
-    if (family) GdipFree (family);
+    GdipFree (family);
 }
 
 




More information about the wine-cvs mailing list