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

List:       wine-devel
Subject:    Re: mshtml: Added IHTMLTableRow::insertCell method implementation. (try 3)
From:       Jacek Caban <jacek () codeweavers ! com>
Date:       2014-05-26 12:47:46
Message-ID: 538337F2.5080104 () codeweavers ! com
[Download RAW message or body]

Hi Zhenbo,

This version looks good except:

On 05/24/14 11:56, Zhenbo Li wrote:
> +    *row = (IDispatch *)&elem->IHTMLElement_iface;
> +    IDispatch_AddRef(*row);
> +    IHTMLElement_Release(&elem->IHTMLElement_iface);

You don't need to add such AddRef()/Release() calls. Reference count
doesn't change anyway.

Cheers,
Jacek


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