From 5dd6d798a46d6748907137038184acf17a27ef26 Mon Sep 17 00:00:00 2001
From: Rob Shearman <robertshearman@gmail.com>
Date: Sun, 17 Aug 2008 18:32:16 +0100
Subject: [PATCH] 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 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/urlmon/umon.c b/dlls/urlmon/umon.c
index f6ea4274f03..d6a42429821 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;