commit 5ad7d858e0d6168429fca902500ac5b50f14dc30
author Michael Stefaniuc <mstefani@redhat.de> Thu Dec 23 17:06:43 2004 +0000
committer Alexandre Julliard <julliard@winehq.org> Thu Dec 23 17:06:43 2004 +0000
tree 5801e7f60e1982dfd7a54c450c5ab3c015cb8e6a
parent bf42021e06b6757db5b014a8cde443aada22a0a1

Do not check for non NULL pointer before HeapFree'ing it. It's
redundant.

programs/winecfg/winecfg.c

diff --git a/programs/winecfg/winecfg.c b/programs/winecfg/winecfg.c
index f9e630d..a97d527 100644
--- a/programs/winecfg/winecfg.c
+++ b/programs/winecfg/winecfg.c
@@ -222,7 +222,7 @@
 
     HeapFree(GetProcessHeap(), 0, setting->path);
     HeapFree(GetProcessHeap(), 0, setting->name);
-    if (setting->value) HeapFree(GetProcessHeap(), 0, setting->value);
+    HeapFree(GetProcessHeap(), 0, setting->value);
 
     list_remove(&setting->entry);
 
@@ -298,7 +298,7 @@
         if (strcasecmp(s->name, name) != 0) continue;
 
         /* yes, we have already set it, so just replace the content and return  */
-        if (s->value) HeapFree(GetProcessHeap(), 0, s->value);
+        HeapFree(GetProcessHeap(), 0, s->value);
         s->value = value ? strdupA(value) : NULL;
 
         return;
@@ -478,7 +478,7 @@
 {
     static char *result = NULL;
 
-    if (result) HeapFree(GetProcessHeap(), 0, result);
+    HeapFree(GetProcessHeap(), 0, result);
 
     if (current_app)
     {