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