Re: [PATCH 0/6] MR846: uiautomationcore: Implement basic HWND based HUIANODE behavior. (v2)

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/uiautomationcore/uia_provider.c:
> - default:
> -
>    break;
> - }
> -
> - return DefWindowProcW(hwnd, msg, wparam, lparam);
>
> +}
> -
>
> +static DWORD WINAPI uia_provider_thread_proc(void *arg)
> +{
> - HANDLE initialized_event = arg;
> - HWND hwnd;
> - MSG msg;
> -
> - CoInitializeEx(NULL, COINIT_MULTITHREADED);
> - hwnd = CreateWindowW(L"Message", NULL, 0, 0, 0, 0, 0, HWND_MESSAGE, NULL, NULL, NULL);
> - if (!IsWindow(hwnd))

I don't think there's a need to check IsWindow here? If CreateWindow fails, it should return NULL.