Rob Shearman : urlmon: Fix testing of HRESULT types with not operator instead of comparing against S_OK .

Alexandre Julliard julliard at winehq.org
Mon Aug 18 10:47:22 CDT 2008


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

Author: Rob Shearman <robertshearman at gmail.com>
Date:   Sun Aug 17 18:32:16 2008 +0100

urlmon: Fix testing of HRESULT types with not operator instead of comparing against S_OK.

This makes it more obvious what the code is doing.

---

 dlls/urlmon/umon.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/urlmon/umon.c b/dlls/urlmon/umon.c
index f6ea427..d6a4242 100644
--- a/dlls/urlmon/umon.c
+++ b/dlls/urlmon/umon.c
@@ -189,7 +189,7 @@ static HRESULT Binding_MoreCacheData(Binding *This, const char *buf, DWORD dwByt
 					        BINDSTATUS_BEGINDOWNLOADDATA :
 					        BINDSTATUS_DOWNLOADINGDATA,
 					    This->URLName);
-	if (!hr)
+	if (hr == S_OK)
 	{
 	    STGMEDIUM stg;
 	    FORMATETC fmt;




More information about the wine-cvs mailing list