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.