Rob Shearman : quartz: 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:21 CDT 2008


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

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

quartz: 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/quartz/control.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/quartz/control.c b/dlls/quartz/control.c
index e719831..a2db24b 100644
--- a/dlls/quartz/control.c
+++ b/dlls/quartz/control.c
@@ -254,7 +254,7 @@ static HRESULT ForwardCmdSeek( PCRITICAL_SECTION crit_sect, IBaseFilter* from, S
                     IMediaSeeking *seek = NULL;
 
                     hr_local = IPin_QueryInterface( connected, &IID_IMediaSeeking, (void**)&seek );
-                    if (!hr_local)
+                    if (hr_local == S_OK)
                     {
                         foundend = TRUE;
                         if (crit_sect)




More information about the wine-cvs mailing list