Jacek Caban (@jacek) commented about dlls/mshtml/htmlanchor.c:
-static HRESULT HTMLAnchorElement_QI(HTMLDOMNode *iface, REFIID riid, void **ppv) +static void *HTMLAnchorElement_QI(HTMLDOMNode *iface, REFIID riid) { HTMLAnchorElement *This = impl_from_HTMLDOMNode(iface);
- *ppv = NULL; - if(IsEqualGUID(&IID_IUnknown, riid)) { - TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv); - *ppv = &This->IHTMLAnchorElement_iface; + TRACE("(%p)->(IID_IUnknown)\n", This); + return &This->IHTMLAnchorElement_iface; }else if(IsEqualGUID(&IID_IDispatch, riid)) { - TRACE("(%p)->(IID_IDispatch %p)\n", This, ppv); - *ppv = &This->IHTMLAnchorElement_iface; + TRACE("(%p)->(IID_IDispatch)\n", This); Let's get rid of those traces, we have `debugstr_mshtml_guid` now so they are no longer needed.
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/3723#note_44193