Re: [PATCH v13 0/5] MR8829: windows.media.playback.mediaplayer: Implement IActivationFactory::ActivateInstance().

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/windows.media.playback.backgroundmediaplayer/tests/playback.c:
> +#include "windows.foundation.h"
> +#define WIDL_using_Windows_Media_Playback
> +#include "windows.media.playback.h"
> -
>
> +#include "wine/test.h"
> -
>
> +#define check_interface( obj, iid ) check_interface_( __LINE__, obj, iid )
> +static void check_interface_( unsigned int line, void *obj, const IID *iid )
> +{
> - IUnknown *iface = obj;
> - IUnknown *unk;
> - HRESULT hr;
> -
> - hr = IUnknown_QueryInterface( iface, iid, (void **)&unk );
> - ok_(__FILE__, line)( hr == S_OK, "got hr %#lx.\n", hr );
> - IUnknown_Release( unk );

This shouldn't release unconditionally, we don't want to crash if something is missing.