Re: [PATCH 0/1] MR8063: shell32: Create an internal IDropSource in SHDoDragDrop if it wasn't passed by the caller

[Alfred Agrell (＠Alcaro)]

Alfred Agrell (@Alcaro) commented about dlls/shell32/shellord.c:
> - LONG Ref;
>
> +} IDummySource;
> -
>
> +static inline IDummySource *impl_from_IDropSource(IDropSource *iface)
> +{
> - return CONTAINING_RECORD(iface, IDummySource, IDropSource_iface);
>
> +}
> -
>
> +static HRESULT WINAPI IDummyDropSource_QueryInterface(IDropSource *iface, REFIID riid, void **ppvObj)
> +{
> - IDummySource *This = impl_from_IDropSource(iface);
> - TRACE("(%p)->(%s, %p)\n", This, debugstr_guid(riid), ppvObj);
> -
> - *ppvObj = NULL;
> -
> - if(IsEqualIID(riid, &IID_IDropSource))

Shouldn't this also accept IUnknown?