Lucas Fialho Zawacki : dinput: Made EnumDevicesBySemantics more terse.
Alexandre Julliard
julliard at winehq.org
Mon Jul 11 13:17:49 CDT 2011
Module: wine
Branch: master
Commit: 13421a210b0c6d1b2f97b670744e37b6380945bc
URL: http://source.winehq.org/git/wine.git/?a=commit;h=13421a210b0c6d1b2f97b670744e37b6380945bc
Author: Lucas Fialho Zawacki <lfzawacki at gmail.com>
Date: Fri Jul 8 15:00:40 2011 -0300
dinput: Made EnumDevicesBySemantics more terse.
---
dlls/dinput/dinput_main.c | 79 +++++++++++++++++++--------------------------
1 files changed, 33 insertions(+), 46 deletions(-)
diff --git a/dlls/dinput/dinput_main.c b/dlls/dinput/dinput_main.c
index 7b46839..207d559 100644
--- a/dlls/dinput/dinput_main.c
+++ b/dlls/dinput/dinput_main.c
@@ -842,12 +842,13 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics(
LPVOID pvRef, DWORD dwFlags
)
{
+ static const REFGUID guids[2] = { &GUID_SysKeyboard, &GUID_SysMouse };
+ static const DWORD actionMasks[] = { DIKEYBOARD_MASK, DIMOUSE_MASK };
IDirectInputImpl *This = impl_from_IDirectInput8A( iface );
DIDEVICEINSTANCEA didevi;
LPDIRECTINPUTDEVICE8A lpdid;
- BOOL ret;
- DWORD callbackFlags = 0;
- int i;
+ int i, j;
+
FIXME("(this=%p,%s,%p,%p,%p,%04x): semi-stub\n", This, ptszUserName, lpdiActionFormat,
lpCallback, pvRef, dwFlags);
@@ -864,29 +865,22 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics(
didevi.dwSize = sizeof(didevi);
- /* enum the keyboard first */
- IDirectInput_CreateDevice(iface, &GUID_SysKeyboard, &lpdid, NULL);
- IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* if there's any DIKEYBOARD action, keyboard is priority 1*/
- for(i=0; i < lpdiActionFormat->dwActionSize; i++)
- if ((lpdiActionFormat->rgoAction[i].dwSemantic & DIKEYBOARD_MASK) == DIKEYBOARD_MASK)
- callbackFlags |= DIEDBS_MAPPEDPRI1;
-
- ret = lpCallback(&didevi, lpdid, callbackFlags, 1, pvRef);
-
- if (ret == DIENUM_STOP) return DI_OK;
+ /* Enumerate keyboard and mouse */
+ for(i=0; i < sizeof(guids)/sizeof(guids[0]); i++)
+ {
+ DWORD callbackFlags = 0;
- callbackFlags = 0;
+ IDirectInput_CreateDevice(iface, guids[i], &lpdid, NULL);
+ IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* and then the mouse */
- IDirectInput_CreateDevice(iface, &GUID_SysMouse, &lpdid, NULL);
- IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* same as above, but for the mouse */
- for(i=0; i < lpdiActionFormat->dwActionSize; i++)
- if ((lpdiActionFormat->rgoAction[i].dwSemantic & DIMOUSE_MASK) == DIMOUSE_MASK)
- callbackFlags |= DIEDBS_MAPPEDPRI1;
+ /* If there's at least one action for the device it's priority 1 */
+ for(j=0; j < lpdiActionFormat->dwActionSize; j++)
+ if ((lpdiActionFormat->rgoAction[j].dwSemantic & actionMasks[j]) == actionMasks[j])
+ callbackFlags |= DIEDBS_MAPPEDPRI1;
- lpCallback(&didevi, lpdid, callbackFlags, 0, pvRef);
+ if (lpCallback(&didevi, lpdid, callbackFlags, sizeof(guids)/sizeof(guids[0]) - (i+1), pvRef) == DIENUM_STOP)
+ return DI_OK;
+ }
return DI_OK;
}
@@ -897,41 +891,34 @@ static HRESULT WINAPI IDirectInput8WImpl_EnumDevicesBySemantics(
LPVOID pvRef, DWORD dwFlags
)
{
+ static const REFGUID guids[2] = { &GUID_SysKeyboard, &GUID_SysMouse };
+ static const DWORD actionMasks[] = { DIKEYBOARD_MASK, DIMOUSE_MASK };
IDirectInputImpl *This = impl_from_IDirectInput8W(iface);
DIDEVICEINSTANCEW didevi;
LPDIRECTINPUTDEVICE8W lpdid;
- BOOL ret;
- DWORD callbackFlags = 0;
- int i;
+ int i, j;
FIXME("(this=%p,%s,%p,%p,%p,%04x): semi-stub\n", This, debugstr_w(ptszUserName), lpdiActionFormat,
lpCallback, pvRef, dwFlags);
didevi.dwSize = sizeof(didevi);
- /* enum the keyboard first */
- IDirectInput_CreateDevice(iface, &GUID_SysKeyboard, &lpdid, NULL);
- IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* if there's any DIKEYBOARD action, keyboard is priority 1*/
- for(i=0; i < lpdiActionFormat->dwActionSize; i++)
- if ((lpdiActionFormat->rgoAction[i].dwSemantic & DIKEYBOARD_MASK) == DIKEYBOARD_MASK)
- callbackFlags |= DIEDBS_MAPPEDPRI1;
-
- ret = lpCallback(&didevi, lpdid, callbackFlags, 1, pvRef);
-
- if (ret == DIENUM_STOP) return DI_OK;
+ /* Enumerate keyboard and mouse */
+ for(i=0; i < sizeof(guids)/sizeof(guids[0]); i++)
+ {
+ DWORD callbackFlags = 0;
- callbackFlags = 0;
+ IDirectInput_CreateDevice(iface, guids[i], &lpdid, NULL);
+ IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* and then the mouse */
- IDirectInput_CreateDevice(iface, &GUID_SysMouse, &lpdid, NULL);
- IDirectInputDevice_GetDeviceInfo(lpdid, &didevi);
- /* same as above, but for the mouse */
- for(i=0; i < lpdiActionFormat->dwActionSize; i++)
- if ((lpdiActionFormat->rgoAction[i].dwSemantic & DIMOUSE_MASK) == DIMOUSE_MASK)
- callbackFlags |= DIEDBS_MAPPEDPRI1;
+ /* If there's at least one action for the device it's priority 1 */
+ for(j=0; j < lpdiActionFormat->dwActionSize; j++)
+ if ((lpdiActionFormat->rgoAction[j].dwSemantic & actionMasks[j]) == actionMasks[j])
+ callbackFlags |= DIEDBS_MAPPEDPRI1;
- lpCallback(&didevi, lpdid, callbackFlags, 0, pvRef);
+ if (lpCallback(&didevi, lpdid, callbackFlags, sizeof(guids)/sizeof(guids[0]) - (i+1), pvRef) == DIENUM_STOP)
+ return DI_OK;
+ }
return DI_OK;
}
More information about the wine-cvs
mailing list