Re: [PATCH v3 0/5] MR9355: windows.perception.stub: Implement IHolographicSpace_get_PrimaryAdapterId().

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.perception.stub/holographic_space.c:
> - holographic_space_CreateNextFrame,
>
> +};
> -
>
> +DEFINE_IINSPECTABLE( holographic_space_interop, IHolographicSpaceInterop, struct holographic_space_statics, IActivationFactory_iface )
> -
>
> +static HRESULT WINAPI holographic_space_interop_CreateForWindow( IHolographicSpaceInterop *iface,
>                                                                   HWND window, REFIID iid, void **holographic_space )
>  {
> - FIXME( "iface %p, window %p, iid %s, holographic_space %p.\n", iface, window, debugstr_guid( iid ), holographic_space );
> -
>
> struct holographic_space *impl;
> -
>
> HRESULT hr;
> -
> -
>
> TRACE( "iface %p, window %p, iid %s, holographic_space %p.\n", iface, window, debugstr_guid( iid ), holographic_space );
> -
>
> if (!window) return E_INVALIDARG;
> -
>
> if (!IsWindow( window ))

I think IsWindow will fail as well if handle is NULL, no need for two separate conditions (and I don't think we really care if one sets NULL and not the other).