Re: [PATCH 0/4] MR5156: mshtml: Always use the event target dispex

[Gabriel Ivăncescu]

Gabriel Ivăncescu (@insn) commented about dlls/mshtml/tests/dom.c:
> }
>
> hres = IHTMLDocument2_QueryInterface(doc, &IID_IEventTarget, (void**)&event_target);
> -    todo_wine
>      ok(hres == S_OK, "QueryInterface(IID_IEventTarget) returned %08lx.\n", hres);
> -    todo_wine
>      ok(event_target != NULL, "event_target == NULL\n");
>      if (event_target != NULL)

Nit: You can remove the if condition now and unconditionally release it, since it should never fail now.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/5156#note_62657