Minor dlls/mshtml/htmlbody.c simplification (RESEND)
Nikolay Sivov
nsivov at codeweavers.com
Sat May 1 03:43:38 CDT 2010
On 5/1/2010 04:04, Gerald Pfeifer wrote:
> On Wed, 28 Apr 2010, Nikolay Sivov wrote:
>
>> Please follow wine-devel comments:
>>
>> http://www.winehq.org/pipermail/wine-devel/2010-April/083175.html
>>
> Ouch, I had missed this message -- certainly did not mean to ignore
> the good feedback.
>
> Is the following what you had in mind?
>
> Gerald
>
>
> ChangeLog:
> Fix return value for HTMLBodyElement_get_text.
>
> ---
> dlls/mshtml/htmlbody.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/mshtml/htmlbody.c b/dlls/mshtml/htmlbody.c
> index 5c66e65..65aa549 100644
> --- a/dlls/mshtml/htmlbody.c
> +++ b/dlls/mshtml/htmlbody.c
> @@ -443,7 +443,7 @@ static HRESULT WINAPI HTMLBodyElement_get_text(IHTMLBodyElement *iface, VARIANT
> HTMLBodyElement *This = HTMLBODY_THIS(iface);
> nsAString text;
> nsresult nsres;
> - HRESULT hres;
> + HRESULT hres = S_OK;
>
> TRACE("(%p)->(%p)\n", This, p);
>
> @@ -462,7 +462,7 @@ static HRESULT WINAPI HTMLBodyElement_get_text(IHTMLBodyElement *iface, VARIANT
>
> nsAString_Finish(&text);
>
> - return S_OK;
> + return hres;
> }
>
> static HRESULT WINAPI HTMLBodyElement_put_link(IHTMLBodyElement *iface, VARIANT v)
>
Yes, but you don't have to initialize it to S_OK.
More information about the wine-devel
mailing list