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

[Bernhard Kölbl (＠besentv)]

Bernhard Kölbl (@besentv) commented about dlls/threadpoolwinrt/main.c:
> - IActivationFactory IActivationFactory_iface;
> - IThreadPoolStatics IThreadPoolStatics_iface;
> - LONG refcount;
>
> +};
> -
>
> +static inline struct threadpool_factory *impl_from_IActivationFactory(IActivationFactory *iface)
> +{
> - return CONTAINING_RECORD(iface, struct threadpool_factory, IActivationFactory_iface);
>
> +}
> -
>
> +static inline struct threadpool_factory *impl_from_IThreadPoolStatics(IThreadPoolStatics *iface)
> +{
> - return CONTAINING_RECORD(iface, struct threadpool_factory, IThreadPoolStatics_iface);
>
> +}
> -
>
> +static HRESULT STDMETHODCALLTYPE threadpool_factory_QueryInterface(

We usually apply WINAPI instead of STDMETHODCALLTYPE to WinRT functions to reduce line length.
Also in most cases you could put the arguments on the same line as the function name.

(Additionally, we kinda agreed on using ntdll style (spaces after function braces), but that isn't so necessary.)