[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!
Vitaliy.
More information about the wine-devel
mailing list