dinput: Support IUnknown in the IDirectInputEffect COM object.

[Huw Davies]

On Tue, Mar 15, 2016 at 11:08:16AM +0100, Michael Stefaniuc wrote:
> On 03/15/2016 10:54 AM, Huw Davies wrote:
> > On Tue, Mar 15, 2016 at 10:37:16AM +0100, Michael Stefaniuc wrote:
> >> Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
> >> ---
> >>  dlls/dinput/joystick_osx.c | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/dlls/dinput/joystick_osx.c b/dlls/dinput/joystick_osx.c
> >> index 6a8926d..19ca070 100644
> >> --- a/dlls/dinput/joystick_osx.c
> >> +++ b/dlls/dinput/joystick_osx.c
> >> @@ -1474,7 +1474,7 @@ static HRESULT WINAPI effect_QueryInterface(IDirectInputEffect *iface,
> >>  
> >>      TRACE("%p %s %p\n", This, debugstr_guid(guid), out);
> >>  
> >> -    if(IsEqualIID(guid, &IID_IDirectInputEffect)){
> >> +    if(IsEqualGUID(riid, &IID_IUnknown) || IsEqualIID(guid, &IID_IDirectInputEffect)){
> > 
> > It looks a bit strange to mix IsEqualGUID and IsEqualIID.
> Duh, missed that. Copy and paste for the win!
> Which one do we prefer? Not that it really matters.

I slightly prefer IsEqualIID for iids, but I don't think we should go
around changing them all.

Huw.