[PATCH 0/16] MR3773: mshtml: Implement a unified QueryInterface on the dispex, part 2.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmlevent.c:
>  return CONTAINING_RECORD(iface, DOMEvent, dispex);
>
> }
>
> +static void *DOMEvent_query_interface(DispatchEx *dispex, REFIID riid)
> +{
> - DOMEvent *This = DOMEvent_from_DispatchEx(dispex);
> -
> - if(IsEqualGUID(&IID_IDOMEvent, riid))
> -
>    return &This->IDOMEvent_iface;
> - if(This->query_interface)

This seems like a good opportunity to get rid of query_interface from DOMEvent object. Anyway, this MR has too many commit, please split it.

[PATCH 0/13] MR3773: mshtml: Implement a unified QueryInterface on the dispex, part 2.

[Gabriel Ivăncescu (＠insn)]

On Fri Sep  8 14:52:15 2023 +0000, Jacek Caban wrote:
> This seems like a good opportunity to get rid of query_interface from
> DOMEvent object. Anyway, this MR has too many commit, please split it.

Oh, I was going to do that on follow up MR. But I'll split it anyway.

[PATCH 0/13] MR3773: mshtml: Implement a unified QueryInterface on the dispex, part 2.

[Jacek Caban (＠jacek)]

On Fri Sep  8 15:23:06 2023 +0000, Gabriel Ivăncescu wrote:
> Oh, I was going to do that on follow up MR. But I'll split it anyway.

I mean, it's fine to do all event objects in a single commit. But this MR has too many commits in total already.