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

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


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

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

msxml3: Remove redundant NULL check before SysFreeString.

---

 dlls/msxml3/saxreader.c |   10 ++++------
 1 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/dlls/msxml3/saxreader.c b/dlls/msxml3/saxreader.c
index 84dd9f4..646df91 100644
--- a/dlls/msxml3/saxreader.c
+++ b/dlls/msxml3/saxreader.c
@@ -1517,10 +1517,8 @@ static ULONG WINAPI isaxlocator_Release(
     ref = InterlockedDecrement( &This->ref );
     if ( ref == 0 )
     {
-        if(This->publicId)
-            SysFreeString(This->publicId);
-        if(This->systemId)
-            SysFreeString(This->systemId);
+        SysFreeString(This->publicId);
+        SysFreeString(This->systemId);
         HeapFree(GetProcessHeap(), 0, This->nsStack);
 
         ISAXXMLReader_Release((ISAXXMLReader*)&This->saxreader->lpSAXXMLReaderVtbl);
@@ -1558,7 +1556,7 @@ static HRESULT WINAPI isaxlocator_getPublicId(
     BSTR publicId;
     saxlocator *This = impl_from_ISAXLocator( iface );
 
-    if(This->publicId) SysFreeString(This->publicId);
+    SysFreeString(This->publicId);
 
     publicId = bstr_from_xmlChar(xmlSAX2GetPublicId(This->pParserCtxt));
     if(SysStringLen(publicId))
@@ -1580,7 +1578,7 @@ static HRESULT WINAPI isaxlocator_getSystemId(
     BSTR systemId;
     saxlocator *This = impl_from_ISAXLocator( iface );
 
-    if(This->systemId) SysFreeString(This->systemId);
+    SysFreeString(This->systemId);
 
     systemId = bstr_from_xmlChar(xmlSAX2GetSystemId(This->pParserCtxt));
     if(SysStringLen(systemId))




More information about the wine-cvs mailing list