Damjan Jovanovic : mshtml/tests: Fix a HRESULT comparison in ProtocolSink_ReportResult().
Alexandre Julliard
julliard at winehq.org
Mon Apr 13 15:05:57 CDT 2020
Module: wine
Branch: master
Commit: 5e75310837e5ec77ebc361d689ea3279fa49ebac
URL: https://source.winehq.org/git/wine.git/?a=commit;h=5e75310837e5ec77ebc361d689ea3279fa49ebac
Author: Damjan Jovanovic <damjan.jov at gmail.com>
Date: Fri Apr 10 13:30:22 2020 +0200
mshtml/tests: Fix a HRESULT comparison in ProtocolSink_ReportResult().
ProtocolSink_ReportResult() only checks expect_hrResult is non-zero,
instead of comparing it against hres
like the same code snippet in test_protocol_fail().
Signed-off-by: Damjan Jovanovic <damjan.jov at gmail.com>
Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/mshtml/tests/protocol.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dlls/mshtml/tests/protocol.c b/dlls/mshtml/tests/protocol.c
index 8f37d09c55..ba767d21c6 100644
--- a/dlls/mshtml/tests/protocol.c
+++ b/dlls/mshtml/tests/protocol.c
@@ -139,7 +139,7 @@ static HRESULT WINAPI ProtocolSink_ReportResult(IInternetProtocolSink *iface, HR
CHECK_EXPECT(ReportResult);
if(expect_hr_win32err) {
- ok((hrResult&0xffff0000) == ((FACILITY_WIN32 << 16)|0x80000000) || expect_hrResult,
+ ok((hrResult&0xffff0000) == ((FACILITY_WIN32 << 16)|0x80000000) || hrResult == expect_hrResult,
"expected win32 err or %08x got: %08x\n", expect_hrResult, hrResult);
}else {
ok(hrResult == expect_hrResult || (expect_hrResult == E_INVALIDARG && hrResult == MK_E_SYNTAX)
More information about the wine-cvs
mailing list