mshtml: Implement HTMLElement_get_children.

Robert Shearman rob at codeweavers.com
Mon Jun 25 09:05:38 CDT 2007


Jacek Caban wrote:
>> +    elem_vector buf = {NULL, 0, 8};
>> +
>> +    TRACE("(%p)->(%p)\n", This, p);
>> +
>> +    buf.buf = mshtml_alloc(buf.size*sizeof(HTMLElement**));
>>     
>
>
> elem_vector is a self-resizable table, but in this case you know the size while creating it from nsIDOMNodeList::GetLength. I'd suggest to use it here during creation of buf to avoid useless memory reallocations.

I've just sent a new patch which includes this suggestion.

-- 
Rob Shearman




More information about the wine-devel mailing list