[1/4] msxml3: Added common ISupportErrorInfo implementation

Jacek Caban jacek at codeweavers.com
Mon Nov 21 07:39:18 CST 2011

On 11/21/11 15:33, Nikolay Sivov wrote:
> On 11/21/2011 16:29, Jacek Caban wrote:
>> On 11/21/11 15:16, Nikolay Sivov wrote:
>>> On 11/21/2011 16:09, Jacek Caban wrote:
>>>> This doesn't seem to be the right approach. You need to return here 
>>>> interfaces of an object that inherits your ISupportErrorInfo 
>>>> implementation - that's the COM rule.
>>> I know but that's not how native works. ISupportErrorInfo reference 
>>> counting is completely separated from object your queried it from.
>> Still, QI doesn't look right (unless you also have tests for this). 
>> See something like this:
>> IDOMXMLNode_QueryInterface(node, &IID_ISupportErrorInfo, (void**)&sei);
>> ISupportErrorInfo_QueryInterface(sei, &IID_IDOMXMLNode, (void**)&node2);
>> The second QueryInterface call should succeeded, but it won't with 
>> your implementation.
> Yeah, I thought about that when I added this implementation, and maybe 
> it should work this way but it doesn't on native :)
> I have a test for that that I'll submit as a separate patch. This 
> interface looks more like meta thing that doesn't belong to any DOM 
> object at all.

OK, the patch is fine then. Thanks for checking it.


More information about the wine-devel mailing list