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

[Bernhard Kölbl (＠besentv)]

Bernhard Kölbl (@besentv) commented about dlls/windows.media/main.c:
> -
>
> +WINE_DEFAULT_DEBUG_CHANNEL(media);
> -
>
> +HRESULT WINAPI DllGetClassObject(REFCLSID clsid, REFIID riid, void **out)
> +{
> - FIXME("(clsid %s, riid %s, out %p) - stub.\n", debugstr_guid(clsid), debugstr_guid(riid), out);
> - return CLASS_E_CLASSNOTAVAILABLE;
>
> +}
> -
>
> +HRESULT WINAPI DllGetActivationFactory(HSTRING classid, IActivationFactory **factory)
> +{
> - TRACE( "(class %s, factory %p.)\n", debugstr_hstring(classid), factory );
> -
> - *factory = NULL;
> -
> - if (*factory) return S_OK;

You're setting *factory to Null in the last statement and then check here if it's unequal 0 to return S_OK.