Re: [PATCH v2 0/5] MR3723: mshtml: Implement a unified QueryInterface on the dispex, part 1.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/xmlhttprequest.c:
> return CONTAINING_RECORD(iface, HTMLXMLHttpRequest, event_target.dispex);
>  }
>
> +static void *HTMLXMLHttpRequest_query_interface(DispatchEx *dispex, REFIID riid)
> +{
> +    HTMLXMLHttpRequest *This = impl_from_DispatchEx(dispex);
> +
> +    TRACE("(%p)->(%s)\n", This, debugstr_mshtml_guid(riid));

It looks mostly good to me now, but can we move traces like this to dispex_query_interface as well? I know that it's useful to know what kind of object it's called on, but we could just print the name from dispex_static_data_t there. It may be convenient to add it to more dispex.c traces, BTW.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3723#note_44495