Sergey Khodych wrote:
> 
> 
> ------------------------------------------------------------------------
> 
> 

> +static HRESULT WINAPI SysKeyboardAImpl_GetProperty(
> ...
> +    if (TRACE_ON(dinput))
> +        _dump_DIPROPHEADER(pdiph);
Please drop the TRACE_ON check - it's being done in dump function.

> ...
> +                hr = SysKeyboardWImpl_GetObjectInfo( 
> (LPDIRECTINPUTDEVICE8W)iface , &didoi,
> +                                                     ps->diph.dwObj, 
> ps->diph.dwHow );

You mixing unicode and ascii. Also if you are implementing 
SysKeyboardAImpl_GetProperty you should also implement it's unicode 
counterpart SysKeyboardWImpl_GetProperty.

And please use 4-space indent.

Vitaliy.


Reply via email to