Re: [PATCH v5 0/3] MR1343: windows.media: Add WinRT Closed Captioning interface.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media/captions.c:
> - str = WindowsGetStringRawBuffer( hstr, &len );
> - return wine_dbgstr_wn( str, len );
>
> +}
> -
>
> +struct captions_statics
> +{
> - IActivationFactory IActivationFactory_iface;
> - LONG ref;
>
> +};
> -
>
> +static inline struct captions_statics *impl_from_IActivationFactory( IActivationFactory *iface )
> +{
> - return CONTAINING_RECORD( iface, struct captions_statics, IActivationFactory_iface );
>
> +}
> -
>
> +static HRESULT WINAPI windows_media_QueryInterface( IActivationFactory *iface, REFIID iid, void **out )

As I believe we're trying to keep one file per class in WinRT, I suggest to prefix these with `factory_` rather than `windows_media_`. Or `captions_factory_` if there's going to be more than the caption class.