Re: [PATCH v2 0/3] MR891: threadpoolwinrt: Initial implementation of RunAsync().

[Bernhard Kölbl (＠besentv)]

Bernhard Kölbl (@besentv) commented about dlls/threadpoolwinrt/tests/threadpool.c:
> +#define check_interface(a, b, c) check_interface_(__LINE__, a, b, c)
> +static void check_interface_(unsigned int line, void *iface_ptr, REFIID iid, BOOL supported)
> +{
> - IUnknown *iface = iface_ptr;
> - HRESULT hr, expected_hr;
> - IUnknown *unk;
> -
> - expected_hr = supported ? S_OK : E_NOINTERFACE;
> -
> - hr = IUnknown_QueryInterface(iface, iid, (void **)&unk);
> - ok_(__FILE__, line)(hr == expected_hr, "Got hr %#lx, expected %#lx.\n", hr, expected_hr);
> - if (SUCCEEDED(hr))
> -
>    IUnknown_Release(unk);
>
> +}
> -
>
> +static const WCHAR *class_name = L"Windows.System.Threading.ThreadPool";

This name is a little ambiguous, given there is also the class ThreadPoolTimer. Maybe `threadpool_class_name`?