[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: mshtml: Added IHTMLInputElement_get_form method implementation. (try 2)
From:       Jacek Caban <jacek () codeweavers ! com>
Date:       2014-07-29 13:25:42
Message-ID: 53D7A0D6.3050806 () codeweavers ! com
[Download RAW message or body]

Hi Zhenbo,

On 07/29/14 05:24, Zhenbo Li wrote:
> +    nsres = nsIDOMHTMLInputElement_GetForm(This->nsinput, &nsform);
> +    if (NS_FAILED(nsres) || nsform == NULL) {
> +        ERR("GetForm failed: %08x, nsform: %p\n", nsres, nsform);
> +        p = NULL;

Not setting output value would be probably fine in this case, but if you
do, please do it right (*p = NULL;).

Jacek


[prev in list] [next in list] [prev in thread] [next in thread]