Detlef Riekenberg : oleaut32: Remove redundant NULL check before SysFreeString.

Alexandre Julliard julliard at winehq.org
Tue Sep 23 09:28:17 CDT 2008


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

Author: Detlef Riekenberg <wine.dev at web.de>
Date:   Sun Sep 21 15:52:16 2008 +0200

oleaut32: Remove redundant NULL check before SysFreeString.

---

 dlls/oleaut32/tmarshal.c |    6 ++--
 dlls/oleaut32/typelib.c  |   55 ++++++++++++++-------------------------------
 2 files changed, 20 insertions(+), 41 deletions(-)

diff --git a/dlls/oleaut32/tmarshal.c b/dlls/oleaut32/tmarshal.c
index fdfe07c..a7b4009 100644
--- a/dlls/oleaut32/tmarshal.c
+++ b/dlls/oleaut32/tmarshal.c
@@ -1420,8 +1420,8 @@ xCall(LPVOID retptr, int method, TMProxyImpl *tpinfo /*, args */)
 	TRACE_(olerelay)("(");
     }
 
-    if (iname) SysFreeString(iname);
-    if (fname) SysFreeString(fname);
+    SysFreeString(iname);
+    SysFreeString(fname);
 
     memset(&buf,0,sizeof(buf));
 
@@ -2080,7 +2080,7 @@ TMStubImpl_Invoke(
         goto exit;
     }
 
-    if (iname) SysFreeString (iname);
+    SysFreeString (iname);
 
     /* Need them for hack below */
     memset(names,0,sizeof(names));
diff --git a/dlls/oleaut32/typelib.c b/dlls/oleaut32/typelib.c
index ceb2295..9539bd0 100644
--- a/dlls/oleaut32/typelib.c
+++ b/dlls/oleaut32/typelib.c
@@ -838,7 +838,7 @@ enddeleteloop:
     }
 
 end:
-    if (tlibPath) SysFreeString(tlibPath);
+    SysFreeString(tlibPath);
     if (typeLib) ITypeLib_Release(typeLib);
     if (subKey) RegCloseKey(subKey);
     if (key) RegCloseKey(key);
@@ -3833,29 +3833,17 @@ static ULONG WINAPI ITypeLib2_fnRelease( ITypeLib2 *iface)
       }
       TRACE(" destroying ITypeLib(%p)\n",This);
 
-      if (This->Name)
-      {
-          SysFreeString(This->Name);
-          This->Name = NULL;
-      }
+      SysFreeString(This->Name);
+      This->Name = NULL;
 
-      if (This->DocString)
-      {
-          SysFreeString(This->DocString);
-          This->DocString = NULL;
-      }
+      SysFreeString(This->DocString);
+      This->DocString = NULL;
 
-      if (This->HelpFile)
-      {
-          SysFreeString(This->HelpFile);
-          This->HelpFile = NULL;
-      }
+      SysFreeString(This->HelpFile);
+      This->HelpFile = NULL;
 
-      if (This->HelpStringDll)
-      {
-          SysFreeString(This->HelpStringDll);
-          This->HelpStringDll = NULL;
-      }
+      SysFreeString(This->HelpStringDll);
+      This->HelpStringDll = NULL;
 
       for (pCustData = This->pCustData; pCustData; pCustData = pCustDataNext)
       {
@@ -4679,23 +4667,14 @@ static ULONG WINAPI ITypeInfo_fnRelease(ITypeInfo2 *iface)
       if (This->no_free_data)
           goto finish_free;
 
-      if (This->Name)
-      {
-          SysFreeString(This->Name);
-          This->Name = 0;
-      }
+      SysFreeString(This->Name);
+      This->Name = NULL;
 
-      if (This->DocString)
-      {
-          SysFreeString(This->DocString);
-          This->DocString = 0;
-      }
+      SysFreeString(This->DocString);
+      This->DocString = NULL;
 
-      if (This->DllName)
-      {
-          SysFreeString(This->DllName);
-          This->DllName = 0;
-      }
+      SysFreeString(This->DllName);
+      This->DllName = NULL;
 
       for (pFInfo = This->funclist; pFInfo; pFInfo = pFInfoNext)
       {
@@ -6526,7 +6505,7 @@ static HRESULT WINAPI ITypeInfo_fnAddressOfMember( ITypeInfo2 *iface,
     {
         ERR("couldn't load %s\n", debugstr_w(dll));
         SysFreeString(dll);
-        if (entry) SysFreeString(entry);
+        SysFreeString(entry);
         return STG_E_FILENOTFOUND;
     }
     /* FIXME: store library somewhere where we can free it */
@@ -6552,7 +6531,7 @@ static HRESULT WINAPI ITypeInfo_fnAddressOfMember( ITypeInfo2 *iface,
     }
 
     SysFreeString(dll);
-    if (entry) SysFreeString(entry);
+    SysFreeString(entry);
 
     if (!*ppv)
         return TYPE_E_DLLFUNCTIONNOTFOUND;




More information about the wine-cvs mailing list