dinput: Support IUnknown in the IDirectInputEffect COM object.

[Michael Stefaniuc]

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.

There is also a similar GUID / IID inconsistency in the QI method
declaration. Some use REFGUID the others REFIID or const GUID*...
PITA for my coccinelle scripts.

bye
	michael