Re: [PATCH v4 1/2] uiautomationcore: Implement UiaHostProviderFromHwnd.

[Nikolay Sivov]

On 9/25/21 2:01 AM, Connor McAdams wrote:
> +ULONG WINAPI HwndProvider_Release(IRawElementProviderSimple *iface)
> +{
> - HwndProvider *This = impl_from_HwndProvider(iface);
> - ULONG ref = InterlockedDecrement(&This->ref);
> -
> - if (!ref)
> -
>    heap_free(This);
> -
> - return ref;
>
> +}

For newly added parts there is an opportunity to use cleaner types and
names from the start, like "struct hwndprovider" and "provider" instead
of non-specific "This".

It's obviously not a blocker, but something to consider.