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

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/mshtml/htmlanchor.c:
> unlink_ref(&This->nsanchor);
>  }
>
> -static HRESULT HTMLAnchorElement_handle_event(DispatchEx *dispex, eventid_t eid, nsIDOMEvent *event, BOOL *prevent_default)
> +static HRESULT HTMLAnchorElement_handle_event(DispatchEx *dispex, DOMEvent *event, BOOL *prevent_default)
>  {
>      HTMLAnchorElement *This = impl_from_DispatchEx(dispex);
> +    eventid_t eid = event->event_id;

Here and in other helpers there are just single uses of `eid`s, please just replace them instead of adding variables. Other than that, it looks good to me.

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