Re: [PATCH v4 0/5] MR2884: uiautomationcore: Add support for creating/destroying HUIAEVENTs.

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/uiautomationcore/uia_client.c:
> +    if (FAILED(hr))
> +        return S_OK;
> +
> +    hr = IRawElementProviderFragment_get_FragmentRoot(elfrag, &elroot);
> +    IRawElementProviderFragment_Release(elfrag);
> +    if (FAILED(hr))
> +        return hr;
> +
> +    if (elroot)
> +    {
> +        IRawElementProviderAdviseEvents *advise_events = NULL;
> +
> +        hr = IRawElementProviderFragmentRoot_QueryInterface(elroot, &IID_IRawElementProviderAdviseEvents,
> +                (void **)&advise_events);
> +        IRawElementProviderFragmentRoot_Release(elroot);
> +        if (advise_events)
> It's a bit odd to not check hr here, afaict line 1870 is a dead assignment.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2884#note_33957