[PATCH] mshtml: Use proper constants as HRESULT return values
Nikolay Sivov
nsivov at codeweavers.com
Tue Sep 13 13:54:20 CDT 2016
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
dlls/mshtml/htmlimg.c | 2 +-
dlls/mshtml/htmlnode.c | 4 ++--
dlls/mshtml/htmlstyle.c | 2 +-
dlls/mshtml/htmlwindow.c | 2 +-
dlls/mshtml/ifacewrap.c | 4 ++--
dlls/mshtml/main.c | 2 +-
6 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/dlls/mshtml/htmlimg.c b/dlls/mshtml/htmlimg.c
index 3f37840..76d5c2e 100644
--- a/dlls/mshtml/htmlimg.c
+++ b/dlls/mshtml/htmlimg.c
@@ -291,7 +291,7 @@ static HRESULT WINAPI HTMLImgElement_put_src(IHTMLImgElement *iface, BSTR v)
if(NS_FAILED(nsres))
ERR("SetSrc failed: %08x\n", nsres);
- return NS_OK;
+ return S_OK;
}
static HRESULT WINAPI HTMLImgElement_get_src(IHTMLImgElement *iface, BSTR *p)
diff --git a/dlls/mshtml/htmlnode.c b/dlls/mshtml/htmlnode.c
index 80964c9..1ba477e 100644
--- a/dlls/mshtml/htmlnode.c
+++ b/dlls/mshtml/htmlnode.c
@@ -1167,10 +1167,10 @@ HRESULT HTMLDOMNode_QI(HTMLDOMNode *This, REFIID riid, void **ppv)
*ppv = &This->IHTMLDOMNode2_iface;
}else if(IsEqualGUID(&IID_nsXPCOMCycleCollectionParticipant, riid)) {
*ppv = &node_ccp;
- return NS_OK;
+ return S_OK;
}else if(IsEqualGUID(&IID_nsCycleCollectionISupports, riid)) {
*ppv = &This->IHTMLDOMNode_iface;
- return NS_OK;
+ return S_OK;
}else if(dispex_query_interface(&This->event_target.dispex, riid, ppv)) {
return *ppv ? S_OK : E_NOINTERFACE;
}else {
diff --git a/dlls/mshtml/htmlstyle.c b/dlls/mshtml/htmlstyle.c
index 0ee848f..7a4ebe3 100644
--- a/dlls/mshtml/htmlstyle.c
+++ b/dlls/mshtml/htmlstyle.c
@@ -494,7 +494,7 @@ static HRESULT get_nsstyle_attr_nsval(nsIDOMCSSStyleDeclaration *nsstyle, stylei
nsAString_Finish(&str_name);
- return NS_OK;
+ return S_OK;
}
static HRESULT nsstyle_to_bstr(const WCHAR *val, DWORD flags, BSTR *p)
diff --git a/dlls/mshtml/htmlwindow.c b/dlls/mshtml/htmlwindow.c
index 407de0c..7009571 100644
--- a/dlls/mshtml/htmlwindow.c
+++ b/dlls/mshtml/htmlwindow.c
@@ -443,7 +443,7 @@ HRESULT get_frame_by_name(HTMLOuterWindow *This, const WCHAR *name, BOOL deep, H
return hres;
*ret = window;
- return NS_OK;
+ return S_OK;
}
static HRESULT WINAPI HTMLWindow2_item(IHTMLWindow2 *iface, VARIANT *pvarIndex, VARIANT *pvarResult)
diff --git a/dlls/mshtml/ifacewrap.c b/dlls/mshtml/ifacewrap.c
index a22a008..f31bb4a 100644
--- a/dlls/mshtml/ifacewrap.c
+++ b/dlls/mshtml/ifacewrap.c
@@ -59,7 +59,7 @@ static HRESULT WINAPI wrapper_QueryInterface(IUnknown *iface, REFIID riid, void
return IUnknown_QueryInterface(This->ref_unk, riid, ppv);
}
-static HRESULT WINAPI wrapper_AddRef(IUnknown *iface)
+static ULONG WINAPI wrapper_AddRef(IUnknown *iface)
{
iface_wrapper_t *This = impl_from_IUnknown(iface);
LONG ref = InterlockedIncrement(&This->ref);
@@ -69,7 +69,7 @@ static HRESULT WINAPI wrapper_AddRef(IUnknown *iface)
return ref;
}
-static HRESULT WINAPI wrapper_Release(IUnknown *iface)
+static ULONG WINAPI wrapper_Release(IUnknown *iface)
{
iface_wrapper_t *This = impl_from_IUnknown(iface);
LONG ref = InterlockedDecrement(&This->ref);
diff --git a/dlls/mshtml/main.c b/dlls/mshtml/main.c
index 8a366bd..2a00aee 100644
--- a/dlls/mshtml/main.c
+++ b/dlls/mshtml/main.c
@@ -382,7 +382,7 @@ HRESULT WINAPI RunHTMLApplication( HINSTANCE hinst, HINSTANCE hPrevInst,
LPSTR szCmdLine, INT nCmdShow )
{
FIXME("%p %p %s %d\n", hinst, hPrevInst, debugstr_a(szCmdLine), nCmdShow );
- return 0;
+ return S_OK;
}
/***********************************************************************
--
2.9.3
More information about the wine-patches
mailing list