[PATCH 3/6] dinput: SetActionMap setting the device buffer.

Vitaliy Margolen wine-devel at kievinfo.com
Mon Jun 20 23:42:33 CDT 2011

On 06/20/2011 04:06 PM, Lucas Fialho Zawacki wrote:
> @@ -1391,9 +1391,24 @@ HRESULT WINAPI IDirectInputDevice8AImpl_SetActionMap(LPDIRECTINPUTDEVICE8A iface
>   						     LPCSTR lpszUserName,
>   						     DWORD dwFlags)
>   {
> +    hr = IDirectInputDevice8_SetActionMap(&This->IDirectInputDevice8W_iface,
> +&diafW, NULL, dwFlags);
You need to call IDirectInputDevice8WImpl_SetActionMap directly here. Also 
please don't wrap lines at 80 characters, you can put all that into one 
line. And please be consistent with spacing around parentheses - either put 
spaces on both sides, or don't put at all.

> +        memset(&dp,0,sizeof(dp));
Please don't needlessly zero structures. You setting most values of it 
anyway. And please don't forget spaces after comas.

Otherwise looks really good!


More information about the wine-devel mailing list