wine-commits August 2012

wine-commits@winehq.org

2 participants
541 discussions

Gerald Pfeifer : dinput: Fix incorrect check in device_disabled_registry.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: 74329496932dbec9ba162522c26f1b67e08bae69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=74329496932dbec9ba162522c… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Sat Aug 25 02:18:14 2012 +0200 dinput: Fix incorrect check in device_disabled_registry. --- dlls/dinput/joystick.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/dinput/joystick.c b/dlls/dinput/joystick.c index 0f7ba19..8c0b5e5 100644 --- a/dlls/dinput/joystick.c +++ b/dlls/dinput/joystick.c @@ -51,8 +51,8 @@ static inline IDirectInputDevice8W *IDirectInputDevice8W_from_impl(JoystickGener BOOL device_disabled_registry(const char* name) { - static const char *disabled_str = "disabled"; - static const char *joystick_key = "Joysticks"; + static const char disabled_str[] = "disabled"; + static const char joystick_key[] = "Joysticks"; char buffer[MAX_PATH]; HKEY hkey, appkey, temp; BOOL do_disable = FALSE; @@ -75,7 +75,7 @@ BOOL device_disabled_registry(const char* name) /* Look for the "controllername"="disabled" key */ if (!get_config_key(hkey, appkey, name, buffer, sizeof(buffer))) - if (!strncmp(disabled_str, buffer, sizeof(disabled_str))) + if (!strcmp(disabled_str, buffer)) { TRACE("Disabling joystick '%s' based on registry key.\n", name); do_disable = TRUE;

1 0

Dmitry Timoshkov : gdiplus: Correct signature/mask info for the GIF codec.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: ac8019df3dd972d0411c28a3ca3ad169522ba8b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac8019df3dd972d0411c28a3c… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Aug 27 12:04:31 2012 +0900 gdiplus: Correct signature/mask info for the GIF codec. --- dlls/gdiplus/image.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/gdiplus/image.c b/dlls/gdiplus/image.c index f40713b..c3a542b 100644 --- a/dlls/gdiplus/image.c +++ b/dlls/gdiplus/image.c @@ -3759,8 +3759,8 @@ static const WCHAR gif_codecname[] = {'B', 'u', 'i','l', 't', '-','i', 'n', ' ', static const WCHAR gif_extension[] = {'*','.','G','I','F',0}; static const WCHAR gif_mimetype[] = {'i','m','a','g','e','/','g','i','f', 0}; static const WCHAR gif_format[] = {'G','I','F',0}; -static const BYTE gif_sig_pattern[4] = "GIF8"; -static const BYTE gif_sig_mask[] = { 0xFF, 0xFF, 0xFF, 0xFF }; +static const BYTE gif_sig_pattern[12] = "GIF87aGIF89a"; +static const BYTE gif_sig_mask[] = { 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF }; static const WCHAR tiff_codecname[] = {'B', 'u', 'i','l', 't', '-','i', 'n', ' ', 'T','I','F','F', 0}; static const WCHAR tiff_extension[] = {'*','.','T','I','F','F',';','*','.','T','I','F',0}; @@ -3847,8 +3847,8 @@ static const struct image_codec codecs[NUM_CODECS] = { /* MimeType */ gif_mimetype, /* Flags */ ImageCodecFlagsDecoder | ImageCodecFlagsSupportBitmap | ImageCodecFlagsBuiltin, /* Version */ 1, - /* SigCount */ 1, - /* SigSize */ 4, + /* SigCount */ 2, + /* SigSize */ 6, /* SigPattern */ gif_sig_pattern, /* SigMask */ gif_sig_mask, },

1 0

Lauri Kenttä : joy.cpl: Make Enable/ Disable buttons wider for translations.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: bc62a82f85cd865d831ef2c7ff713a12283916ee URL: http://source.winehq.org/git/wine.git/?a=commit;h=bc62a82f85cd865d831ef2c7f… Author: Lauri Kenttä <lauri.kentta(a)gmail.com> Date: Sat Aug 25 21:29:19 2012 +0300 joy.cpl: Make Enable/Disable buttons wider for translations. --- dlls/joy.cpl/joy.rc | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/joy.cpl/joy.rc b/dlls/joy.cpl/joy.rc index c4a551d..5be4a8c 100644 --- a/dlls/joy.cpl/joy.rc +++ b/dlls/joy.cpl/joy.rc @@ -33,8 +33,8 @@ STYLE WS_CAPTION | WS_CHILD | WS_DISABLED CAPTION "Joysticks" FONT 8, "Ms Shell Dlg" { - PUSHBUTTON "&Disable", IDC_BUTTONDISABLE, 200, 20, 50, 15 - PUSHBUTTON "&Enable", IDC_BUTTONENABLE, 200, 105, 50, 15 + PUSHBUTTON "&Disable", IDC_BUTTONDISABLE, 200, 20, 60, 15 + PUSHBUTTON "&Enable", IDC_BUTTONENABLE, 200, 105, 60, 15 LTEXT "Connected", IDC_STATIC, 10, 10, 100, 10 LISTBOX IDC_JOYSTICKLIST, 10, 20, 180, 70, WS_TABSTOP | WS_VSCROLL | LBS_NOTIFY LTEXT "Disabled", IDC_STATIC, 10, 95, 100, 10

1 0

Lauri Kenttä : po: Update Finnish translation.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: 2ad570a1fb769b241db53b0e03ab9e9ad1d9973b URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ad570a1fb769b241db53b0e0… Author: Lauri Kenttä <lauri.kentta(a)gmail.com> Date: Sat Aug 25 21:29:18 2012 +0300 po: Update Finnish translation. --- po/fi.po | 17 +++++++++-------- 1 files changed, 9 insertions(+), 8 deletions(-) diff --git a/po/fi.po b/po/fi.po index dfc9e7d..30dee50 100644 --- a/po/fi.po +++ b/po/fi.po @@ -3442,6 +3442,8 @@ msgid "" "After disabling or enabling a device, the connected joysticks won't be " "updated here until you restart this applet." msgstr "" +"Kun laite poistetaan käytöstä tai otetaan käyttöön, liitetyt ohjaimet eivät " +"päivity tähän, ellei tätä sovelmaa käynnistetä uudelleen." #: joy.rc:47 msgid "Test Joystick" @@ -9664,11 +9666,11 @@ msgstr "" "/min Käynnistä ohjelma pienennettynä.\n" "/max Käynnistä ohjelma suurennettuna.\n" "/low Käynnistä ohjelma matalassa prioriteettiluokassa.\n" -"/normal Käynnistä ohjelma normaalissa prioriteetiluokassa.\n" -"/high Käynnistä ohjelma korkeassa prioriteetiluokassa.\n" -"/realtime Käynnistä ohjelma reaaliaikaisessa prioriteetiluokassa.\n" -"/abovenormal Käynnistä ohjelma normaalia korkeammassa prioriteetiluokassa.\n" -"/belownormal Käynnistä ohjelma normaalia matalammassa prioriteetiluokassa.\n" +"/normal Käynnistä ohjelma normaalissa prioriteettiluokassa.\n" +"/high Käynnistä ohjelma korkeassa prioriteettiluokassa.\n" +"/realtime Käynnistä ohjelma reaaliaikaisessa prioriteettiluokassa.\n" +"/abovenormal Käynnistä ohjelma normaalia korkeammassa prioriteettiluokassa.\n" +"/belownormal Käynnistä ohjelma normaalia matalammassa prioriteettiluokassa.\n" "/node n Käynnistä ohjelma määrätyllä NUMA-solmulla.\n" "/affinity peite Käynnistä ohjelma määrätyllä affiniteettipeitteellä.\n" "/wait Odota, että ohjelma sulkeutuu, ja palauta sen paluuarvo.\n" @@ -14136,7 +14138,6 @@ msgstr "" "[/M] Kopioi vain tiedostot, joilla on arkistointiattribuutti, ja\n" "\tpoista arkistointiattribuutti.\n" "[/D | /D:k-p-v] Kopioi uudet tiedostot tai ne, joita on muokattu annetun\n" -"\t\tpäivän jälkeen. Jos päivää ei anneta, kopioi vain, jos kohde on " -"vanhempi\n" -"\t\tkuin lähde.\n" +"\t\tpäivän jälkeen. Jos päivää ei anneta, kopioi vain, jos\n" +"\t\tkohde on vanhempi kuin lähde.\n" "\n"

1 0

André Hentschel : po: Update German translation.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: ecb5c84b8e15cd590fee12d9c4547a4bca3cc00a URL: http://source.winehq.org/git/wine.git/?a=commit;h=ecb5c84b8e15cd590fee12d9c… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Aug 25 13:40:32 2012 +0200 po: Update German translation. --- po/de.po | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/po/de.po b/po/de.po index cf995cd..85d1056 100644 --- a/po/de.po +++ b/po/de.po @@ -3454,6 +3454,8 @@ msgid "" "After disabling or enabling a device, the connected joysticks won't be " "updated here until you restart this applet." msgstr "" +"Nach dem deaktivieren oder aktivieren eines Gerätes, wird der verbundene " +"Controller hier bis zum Neustart des Applets nicht mehr aktualisiert." #: joy.rc:47 msgid "Test Joystick"

1 0

Aurimas Fišeras : po: Update Lithuanian translation.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: 1e86d79036b8f0d9ceedeac45b6e8ac6d5bfcc64 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1e86d79036b8f0d9ceedeac45… Author: Aurimas Fišeras <aurimas(a)gmail.com> Date: Fri Aug 24 22:13:34 2012 +0300 po: Update Lithuanian translation. --- po/lt.po | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/po/lt.po b/po/lt.po index ac8298f..0586696 100644 --- a/po/lt.po +++ b/po/lt.po @@ -5,7 +5,7 @@ msgstr "" "Project-Id-Version: Wine\n" "Report-Msgid-Bugs-To: http://bugs.winehq.org\n" "POT-Creation-Date: N/A\n" -"PO-Revision-Date: 2012-07-17 20:09+0300\n" +"PO-Revision-Date: 2012-08-24 22:13+0300\n" "Last-Translator: Aurimas Fišeras <aurimas(a)members.fsf.org>\n" "Language-Team: komp_lt(a)konf.lt\n" "Language: Lithuanian\n" @@ -3455,6 +3455,8 @@ msgid "" "After disabling or enabling a device, the connected joysticks won't be " "updated here until you restart this applet." msgstr "" +"Po įtaiso išjungimo ar įjungimo, prijungtos vairasvirtės nebus automatiškai " +"čia atnaujintos, kol nepaleisite šios programėlės iš naujo." #: joy.rc:47 msgid "Test Joystick"

1 0

Marek Chmiel : dssenh/tests: Added key exchange tests for the DSSENH cryptographic service provider.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: 97fda5a413aa33d37515fe59be9d8ee4a0ab13bf URL: http://source.winehq.org/git/wine.git/?a=commit;h=97fda5a413aa33d37515fe59b… Author: Marek Chmiel <kcmark(a)gmail.com> Date: Sat Aug 18 00:02:33 2012 -0500 dssenh/tests: Added key exchange tests for the DSSENH cryptographic service provider. --- dlls/dssenh/tests/dssenh.c | 396 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 396 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=97fda5a413aa33d37515f…

1 0

Lucas Zawacki : dinput: SetActionMap saving mappings.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: af1dcde288b96e1e12a76861b78b1801f5d52d67 URL: http://source.winehq.org/git/wine.git/?a=commit;h=af1dcde288b96e1e12a76861b… Author: Lucas Zawacki <lfzawacki(a)gmail.com> Date: Fri Aug 24 14:04:52 2012 -0300 dinput: SetActionMap saving mappings. --- dlls/dinput/device.c | 143 ++++++++++++++++++++++++++++++++++++++++++ dlls/dinput/dinput_private.h | 2 + dlls/dinput8/tests/device.c | 8 +- 3 files changed, 149 insertions(+), 4 deletions(-) diff --git a/dlls/dinput/device.c b/dlls/dinput/device.c index bf5fd1b..098efb7 100644 --- a/dlls/dinput/device.c +++ b/dlls/dinput/device.c @@ -598,10 +598,142 @@ static DWORD semantic_to_obj_id(IDirectInputDeviceImpl* This, DWORD dwSemantic) return type | (0x0000ff00 & (obj_instance << 8)); } +/* + * get_mapping_key + * Retrieves an open registry key to save the mapping, parametrized for an username, + * specific device and specific action mapping guid. + */ +static HKEY get_mapping_key(const WCHAR *device, const WCHAR *username, const WCHAR *guid) +{ + static const WCHAR subkey[] = { + 'S','o','f','t','w','a','r','e','\\', + 'W','i','n','e','\\', + 'D','i','r','e','c','t','I','n','p','u','t','\\', + 'M','a','p','p','i','n','g','s','\\','%','s','\\','%','s','\\','%','s','\0'}; + HKEY hkey; + WCHAR *keyname; + + keyname = HeapAlloc(GetProcessHeap(), 0, + sizeof(WCHAR) * (lstrlenW(subkey) + strlenW(username) + strlenW(device) + strlenW(guid))); + sprintfW(keyname, subkey, username, device, guid); + + /* The key used is HKCU\Software\Wine\DirectInput\Mappings\[username]\[device]\[mapping_guid] */ + if (RegCreateKeyW(HKEY_CURRENT_USER, keyname, &hkey)) + hkey = 0; + + HeapFree(GetProcessHeap(), 0, keyname); + + return hkey; +} + +static HRESULT save_mapping_settings(IDirectInputDevice8W *iface, LPDIACTIONFORMATW lpdiaf, LPCWSTR lpszUsername) +{ + WCHAR *guid_str = NULL; + DIDEVICEINSTANCEW didev; + HKEY hkey; + int i; + + didev.dwSize = sizeof(didev); + IDirectInputDevice8_GetDeviceInfo(iface, &didev); + + if (StringFromCLSID(&lpdiaf->guidActionMap, &guid_str) != S_OK) + return DI_SETTINGSNOTSAVED; + + hkey = get_mapping_key(didev.tszInstanceName, lpszUsername, guid_str); + + if (!hkey) + { + CoTaskMemFree(guid_str); + return DI_SETTINGSNOTSAVED; + } + + /* Write each of the actions mapped for this device. + Format is "dwSemantic"="dwObjID" and key is of type REG_DWORD + */ + for (i = 0; i < lpdiaf->dwNumActions; i++) + { + static const WCHAR format[] = {'%','x','\0'}; + WCHAR label[9]; + + if (IsEqualGUID(&didev.guidInstance, &lpdiaf->rgoAction[i].guidInstance) && + lpdiaf->rgoAction[i].dwHow != DIAH_UNMAPPED) + { + sprintfW(label, format, lpdiaf->rgoAction[i].dwSemantic); + RegSetValueExW(hkey, label, 0, REG_DWORD, (const BYTE*) &lpdiaf->rgoAction[i].dwObjID, sizeof(DWORD)); + } + } + + RegCloseKey(hkey); + CoTaskMemFree(guid_str); + + return DI_OK; +} + +static BOOL load_mapping_settings(IDirectInputDeviceImpl *This, LPDIACTIONFORMATW lpdiaf, const WCHAR *username) +{ + HKEY hkey; + WCHAR *guid_str; + DIDEVICEINSTANCEW didev; + int i; + + didev.dwSize = sizeof(didev); + IDirectInputDevice8_GetDeviceInfo(&This->IDirectInputDevice8W_iface, &didev); + + if (StringFromCLSID(&lpdiaf->guidActionMap, &guid_str) != S_OK) + return FALSE; + + hkey = get_mapping_key(didev.tszInstanceName, username, guid_str); + + if (!hkey) + { + CoTaskMemFree(guid_str); + return FALSE; + } + + /* Try to read each action in the DIACTIONFORMAT from registry */ + for (i = 0; i < lpdiaf->dwNumActions; i++) + { + static const WCHAR format[] = {'%','x','\0'}; + DWORD id, size = sizeof(DWORD); + WCHAR label[9]; + + sprintfW(label, format, lpdiaf->rgoAction[i].dwSemantic); + + if (!RegQueryValueExW(hkey, label, 0, NULL, (LPBYTE) &id, &size)) + { + lpdiaf->rgoAction[i].dwObjID = id; + lpdiaf->rgoAction[i].guidInstance = didev.guidInstance; + lpdiaf->rgoAction[i].dwHow = DIAH_USERCONFIG; + } + } + + RegCloseKey(hkey); + CoTaskMemFree(guid_str); + + return TRUE; +} + HRESULT _build_action_map(LPDIRECTINPUTDEVICE8W iface, LPDIACTIONFORMATW lpdiaf, LPCWSTR lpszUserName, DWORD dwFlags, DWORD devMask, LPCDIDATAFORMAT df) { IDirectInputDeviceImpl *This = impl_from_IDirectInputDevice8W(iface); + WCHAR username[MAX_PATH]; + DWORD username_size = MAX_PATH; int i, has_actions = 0; + BOOL load_success = FALSE; + + /* Unless asked the contrary by these flags, try to load a previous mapping */ + if (!(dwFlags & DIDBAM_HWDEFAULTS)) + { + /* Retrieve logged user name if necessary */ + if (lpszUserName == NULL) + GetUserNameW(username, &username_size); + else + lstrcpynW(username, lpszUserName, MAX_PATH); + + load_success = load_mapping_settings(This, lpdiaf, username); + } + + if (load_success) return DI_OK; for (i=0; i < lpdiaf->dwNumActions; i++) { @@ -650,6 +782,8 @@ HRESULT _set_action_map(LPDIRECTINPUTDEVICE8W iface, LPDIACTIONFORMATW lpdiaf, L DIOBJECTDATAFORMAT *obj_df = NULL; DIPROPDWORD dp; DIPROPRANGE dpr; + WCHAR username[MAX_PATH]; + DWORD username_size = MAX_PATH; int i, action = 0, num_actions = 0; unsigned int offset = 0; @@ -722,6 +856,15 @@ HRESULT _set_action_map(LPDIRECTINPUTDEVICE8W iface, LPDIACTIONFORMATW lpdiaf, L IDirectInputDevice8_SetProperty(iface, DIPROP_BUFFERSIZE, &dp.diph); } + /* Retrieve logged user name if necessary */ + if (lpszUserName == NULL) + GetUserNameW(username, &username_size); + else + lstrcpynW(username, lpszUserName, MAX_PATH); + + /* Save the settings to disk */ + save_mapping_settings(iface, lpdiaf, username); + return IDirectInputDevice8WImpl_SetActionMap(iface, lpdiaf, lpszUserName, dwFlags); } diff --git a/dlls/dinput/dinput_private.h b/dlls/dinput/dinput_private.h index 8efb42e..e593b40 100644 --- a/dlls/dinput/dinput_private.h +++ b/dlls/dinput/dinput_private.h @@ -70,6 +70,8 @@ extern void _copy_diactionformatWtoA(LPDIACTIONFORMATA, LPDIACTIONFORMATW) DECLS extern HRESULT _configure_devices(IDirectInput8W *iface, LPDICONFIGUREDEVICESCALLBACK lpdiCallback, LPDICONFIGUREDEVICESPARAMSW lpdiCDParams, DWORD dwFlags, LPVOID pvRefData) DECLSPEC_HIDDEN; +extern WCHAR* get_mapping_path(const WCHAR *device, const WCHAR *username) DECLSPEC_HIDDEN; + #define IS_DIPROP(x) (((ULONG_PTR)(x) >> 16) == 0) #define DIKEYBOARD_MASK 0x81000000 diff --git a/dlls/dinput8/tests/device.c b/dlls/dinput8/tests/device.c index e5b3f6a..76f9120 100644 --- a/dlls/dinput8/tests/device.c +++ b/dlls/dinput8/tests/device.c @@ -458,13 +458,13 @@ static void test_save_settings(void) hr = IDirectInputDevice8_BuildActionMap(pKey, &af, NULL, 0); ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); - todo_wine ok (other_results[0] == af.rgoAction[0].dwObjID, + ok (other_results[0] == af.rgoAction[0].dwObjID, "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", other_results[0], af.rgoAction[0].dwObjID); - todo_wine ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[0].guidInstance), "Action should be mapped to keyboard\n"); + ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[0].guidInstance), "Action should be mapped to keyboard\n"); - todo_wine ok (other_results[1] == af.rgoAction[1].dwObjID, + ok (other_results[1] == af.rgoAction[1].dwObjID, "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", other_results[1], af.rgoAction[1].dwObjID); - todo_wine ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[1].guidInstance), "Action should be mapped to keyboard\n"); + ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[1].guidInstance), "Action should be mapped to keyboard\n"); } START_TEST(device)

1 0

Lucas Zawacki : dinput8/tests: Added tests for saving and retrieving settings with SetActionMap.
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: wine Branch: master Commit: 77cab1ce989a3831470bbd2795576493e1803b84 URL: http://source.winehq.org/git/wine.git/?a=commit;h=77cab1ce989a3831470bbd279… Author: Lucas Zawacki <lfzawacki(a)gmail.com> Date: Fri Aug 24 14:04:51 2012 -0300 dinput8/tests: Added tests for saving and retrieving settings with SetActionMap. --- dlls/dinput8/tests/device.c | 134 +++++++++++++++++++++++++++++++++++++++++- 1 files changed, 130 insertions(+), 4 deletions(-) diff --git a/dlls/dinput8/tests/device.c b/dlls/dinput8/tests/device.c index 2deb348..e5b3f6a 100644 --- a/dlls/dinput8/tests/device.c +++ b/dlls/dinput8/tests/device.c @@ -132,7 +132,7 @@ static void test_build_action_map( ddi.dwSize = sizeof(ddi); IDirectInputDevice_GetDeviceInfo(lpdid, &ddi); - hr = IDirectInputDevice8_BuildActionMap(lpdid, lpdiaf, NULL, DIDBAM_INITIALIZE); + hr = IDirectInputDevice8_BuildActionMap(lpdid, lpdiaf, NULL, DIDBAM_HWDEFAULTS); ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); actions = lpdiaf->rgoAction; @@ -195,8 +195,8 @@ static BOOL CALLBACK enumeration_callback( } /* Building and setting an action map */ - /* It should not use any pre-stored mappings so we use DIDBAM_INITIALIZE */ - hr = IDirectInputDevice8_BuildActionMap(lpdid, data->lpdiaf, NULL, DIDBAM_INITIALIZE); + /* It should not use any pre-stored mappings so we use DIDBAM_HWDEFAULTS */ + hr = IDirectInputDevice8_BuildActionMap(lpdid, data->lpdiaf, NULL, DIDBAM_HWDEFAULTS); ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); /* Device has no data format and thus can't be acquired */ @@ -322,7 +322,7 @@ static void test_action_mapping(void) af.dwDataSize = 4 * DITEST_KEYBOARDSPACE; af.dwNumActions = DITEST_KEYBOARDSPACE; - hr = IDirectInputDevice8_BuildActionMap(data.keyboard, data.lpdiaf, NULL, DIDBAM_INITIALIZE); + hr = IDirectInputDevice8_BuildActionMap(data.keyboard, data.lpdiaf, NULL, DIDBAM_HWDEFAULTS); ok (hr == DI_NOEFFECT, "BuildActionMap should have no effect with no actions hr=%08x\n", hr); hr = IDirectInputDevice8_SetActionMap(data.keyboard, data.lpdiaf, NULL, 0); @@ -342,11 +342,137 @@ static void test_action_mapping(void) } } +static void test_save_settings(void) +{ + HRESULT hr; + HINSTANCE hinst = GetModuleHandle(NULL); + LPDIRECTINPUT8 pDI = NULL; + DIACTIONFORMAT af; + IDirectInputDevice8A *pKey; + + static const GUID mapping_guid = { 0xcafecafe, 0x2, 0x3, { 0x4, 0x5, 0x6, 0x7, 0x8, 0x9, 0xa, 0xb } }; + static DIACTION actions[] = { + { 0, DIKEYBOARD_A , 0, { "Blam" } }, + { 1, DIKEYBOARD_B , 0, { "Kapow"} } + }; + static const DWORD results[] = { + DIDFT_MAKEINSTANCE(DIK_A) | DIDFT_PSHBUTTON, + DIDFT_MAKEINSTANCE(DIK_B) | DIDFT_PSHBUTTON + }; + static const DWORD other_results[] = { + DIDFT_MAKEINSTANCE(DIK_C) | DIDFT_PSHBUTTON, + DIDFT_MAKEINSTANCE(DIK_D) | DIDFT_PSHBUTTON + }; + + hr = CoCreateInstance(&CLSID_DirectInput8, 0, CLSCTX_INPROC_SERVER, &IID_IDirectInput8A, (LPVOID*)&pDI); + if (hr == DIERR_OLDDIRECTINPUTVERSION || + hr == DIERR_BETADIRECTINPUTVERSION || + hr == REGDB_E_CLASSNOTREG) + { + win_skip("ActionMapping requires dinput8\n"); + return; + } + ok (SUCCEEDED(hr), "DirectInput8 Create failed: hr=%08x\n", hr); + if (FAILED(hr)) return; + + hr = IDirectInput8_Initialize(pDI,hinst, DIRECTINPUT_VERSION); + if (hr == DIERR_OLDDIRECTINPUTVERSION || hr == DIERR_BETADIRECTINPUTVERSION) + { + win_skip("ActionMapping requires dinput8\n"); + return; + } + ok (SUCCEEDED(hr), "DirectInput8 Initialize failed: hr=%08x\n", hr); + if (FAILED(hr)) return; + + hr = IDirectInput_CreateDevice(pDI, &GUID_SysKeyboard, &pKey, NULL); + ok (SUCCEEDED(hr), "IDirectInput_Create device failed hr: 0x%08x\n", hr); + if (FAILED(hr)) return; + + memset (&af, 0, sizeof(af)); + af.dwSize = sizeof(af); + af.dwActionSize = sizeof(DIACTION); + af.dwDataSize = 4 * sizeof(actions) / sizeof(actions[0]); + af.dwNumActions = sizeof(actions) / sizeof(actions[0]); + af.rgoAction = actions; + af.guidActionMap = mapping_guid; + af.dwGenre = 0x01000000; /* DIVIRTUAL_DRIVING_RACE */ + af.dwBufferSize = 32; + + /* Easy case. Ask for default mapping, save, ask for previous map and read it back */ + hr = IDirectInputDevice8_BuildActionMap(pKey, &af, NULL, DIDBAM_HWDEFAULTS); + ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); + + ok (results[0] == af.rgoAction[0].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", results[0], af.rgoAction[0].dwObjID); + + ok (results[1] == af.rgoAction[1].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", results[1], af.rgoAction[1].dwObjID); + + hr = IDirectInputDevice8_SetActionMap(pKey, &af, NULL, DIDSAM_FORCESAVE); + ok (SUCCEEDED(hr), "SetActionMap failed hr=%08x\n", hr); + + if (hr == DI_SETTINGSNOTSAVED) + { + skip ("Cant test saving settings if SetActionMap returns DI_SETTINGSNOTSAVED\n"); + return; + } + + af.rgoAction[0].dwObjID = 0; + af.rgoAction[1].dwObjID = 0; + memset(&af.rgoAction[0].guidInstance, 0, sizeof(GUID)); + memset(&af.rgoAction[1].guidInstance, 0, sizeof(GUID)); + + hr = IDirectInputDevice8_BuildActionMap(pKey, &af, NULL, 0); + ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); + + ok (results[0] == af.rgoAction[0].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", results[0], af.rgoAction[0].dwObjID); + ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[0].guidInstance), "Action should be mapped to keyboard\n"); + + ok (results[1] == af.rgoAction[1].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", results[1], af.rgoAction[1].dwObjID); + ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[1].guidInstance), "Action should be mapped to keyboard\n"); + + /* Hard case. Customized mapping, save, ask for previous map and read it back */ + af.rgoAction[0].dwObjID = other_results[0]; + af.rgoAction[0].dwHow = DIAH_USERCONFIG; + af.rgoAction[0].guidInstance = GUID_SysKeyboard; + af.rgoAction[1].dwObjID = other_results[1]; + af.rgoAction[1].dwHow = DIAH_USERCONFIG; + af.rgoAction[1].guidInstance = GUID_SysKeyboard; + + hr = IDirectInputDevice8_SetActionMap(pKey, &af, NULL, DIDSAM_FORCESAVE); + ok (SUCCEEDED(hr), "SetActionMap failed hr=%08x\n", hr); + + if (hr == DI_SETTINGSNOTSAVED) + { + skip ("Cant test saving settings if SetActionMap returns DI_SETTINGSNOTSAVED\n"); + return; + } + + af.rgoAction[0].dwObjID = 0; + af.rgoAction[1].dwObjID = 0; + memset(&af.rgoAction[0].guidInstance, 0, sizeof(GUID)); + memset(&af.rgoAction[1].guidInstance, 0, sizeof(GUID)); + + hr = IDirectInputDevice8_BuildActionMap(pKey, &af, NULL, 0); + ok (SUCCEEDED(hr), "BuildActionMap failed hr=%08x\n", hr); + + todo_wine ok (other_results[0] == af.rgoAction[0].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", other_results[0], af.rgoAction[0].dwObjID); + todo_wine ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[0].guidInstance), "Action should be mapped to keyboard\n"); + + todo_wine ok (other_results[1] == af.rgoAction[1].dwObjID, + "Mapped incorrectly expected: 0x%08x got: 0x%08x\n", other_results[1], af.rgoAction[1].dwObjID); + todo_wine ok (IsEqualGUID(&GUID_SysKeyboard, &af.rgoAction[1].guidInstance), "Action should be mapped to keyboard\n"); +} + START_TEST(device) { CoInitialize(NULL); test_action_mapping(); + test_save_settings(); CoUninitialize(); }

1 0

Francois Gouget : testbot: lib: Remove an unused variable in Job:: ScheduleOnHost().
by Alexandre Julliard 27 Aug '12

27 Aug '12

Module: tools Branch: master Commit: cb8e3fdbd11aafb7db1f84b5d7362bcc0089a4e9 URL: http://source.winehq.org/git/tools.git/?a=commit;h=cb8e3fdbd11aafb7db1f84b5… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Mon Aug 27 11:50:49 2012 +0200 testbot: lib: Remove an unused variable in Job::ScheduleOnHost(). --- testbot/lib/WineTestBot/Jobs.pm | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm index ab3c5c6..0a9701e 100644 --- a/testbot/lib/WineTestBot/Jobs.pm +++ b/testbot/lib/WineTestBot/Jobs.pm @@ -366,7 +366,6 @@ sub ScheduleOnHost my @SortedJobs = sort CompareJobPriority @{$self->GetItems()}; my $DirtyIndex = 0; - my $RevertPriority; foreach my $Job (@SortedJobs) { my $Steps = $Job->Steps; @@ -387,8 +386,7 @@ sub ScheduleOnHost my $VM = $HostVMs->GetItem($Task->VM->GetKey()); if ($VM->Status eq "idle" && (! defined($MaxRunningVMs) || $RunningVMs < $MaxRunningVMs) && - $RevertingVMs == 0 && - (! defined($RevertPriority) || $Job->Priority <= $RevertPriority)) + $RevertingVMs == 0) { $VM->Status("running"); my ($ErrProperty, $ErrMessage) = $HostVMs->Save();

1 0

← Newer
1
...
8
9
10
11
12
13
14
...
55
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits August 2012