Re: dinput/tests: Initial tests for Action Mapping. Added some tests for the creation of a DirectInput8 Interface and a dummy call of EnumDevicesBySemantics.
25 Apr
2011
25 Apr
'11
11:14 p.m.
On Sun, Apr 24, 2011 at 18:12, Lucas Fialho Zawacki <lfzawacki(a)gmail.com> wrote:
Howdy Lucas,
+ hr = CoCreateInstance(&CLSID_DirectInput8, 0,
1, &IID_IDirectInput8A, (LPVOID*)&pDI);
+ if (hr == DIERR_OLDDIRECTINPUTVERSION ||
+ hr == DIERR_BETADIRECTINPUTVERSION ||
+ hr == REGDB_E_CLASSNOTREG)
+ {
+ 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, 0x0800 /* DirectInput8 */ );
+ if (hr == DIERR_OLDDIRECTINPUTVERSION || hr == DIERR_BETADIRECTINPUTVERSION)
+ {
+ skip("ActionMapping requires dinput8\n");
+ return;
+ }
You probably should use win_skip here, Wine shouldn't have missing
dinput8 support.
Cheers,
Austin
26 Apr
26 Apr
1:54 p.m.
New subject: dinput/tests: Initial tests for Action Mapping. Added some tests for the creation of a DirectInput8 Interface and a dummy call of EnumDevicesBySemantics.
On 04/25/2011 05:14 PM, Austin English wrote:
On Sun, Apr 24, 2011 at 18:12, Lucas Fialho
Zawacki<lfzawacki(a)gmail.com> wrote:
Howdy Lucas,
IMHO these tests should go into dinput8 not dinput if they are testing
dinput8 functionality.
Vitaliy.
+ hr =
CoCreateInstance(&CLSID_DirectInput8, 0, 1,&IID_IDirectInput8A,
(LPVOID*)&pDI);
+ if (hr == DIERR_OLDDIRECTINPUTVERSION ||
+ hr == DIERR_BETADIRECTINPUTVERSION ||
+ hr == REGDB_E_CLASSNOTREG)
+ {
+ 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, 0x0800 /* DirectInput8 */ );
+ if (hr == DIERR_OLDDIRECTINPUTVERSION || hr == DIERR_BETADIRECTINPUTVERSION)
+ {
+ skip("ActionMapping requires dinput8\n");
+ return;
+ }
You probably should use win_skip here, Wine shouldn't have missing
dinput8 support. 
2:02 p.m.
New subject: dinput/tests: Initial tests for Action Mapping. Added some tests for the creation of a DirectInput8 Interface and a dummy call of EnumDevicesBySemantics.
Good point. I'll create the dinput8 tests and move those there.
2011/4/26 Vitaliy Margolen <wine-devel(a)kievinfo.com>om>:
On 04/25/2011 05:14 PM, Austin English wrote:
--
http://www.twitter.com/lfzawacki
http://www.linesocode.wordpress.com
On Sun, Apr 24, 2011 at 18:12, Lucas Fialho Zawacki<lfzawacki(a)gmail.com>
wrote:
Howdy Lucas,
IMHO these tests should go into dinput8 not dinput if they are testing
dinput8 functionality.
Vitaliy.
+ hr =
CoCreateInstance(&CLSID_DirectInput8, 0, 1,&IID_IDirectInput8A,
(LPVOID*)&pDI);
+ if (hr == DIERR_OLDDIRECTINPUTVERSION ||
+ hr == DIERR_BETADIRECTINPUTVERSION ||
+ hr == REGDB_E_CLASSNOTREG)
+ {
+ 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, 0x0800 /* DirectInput8 */
);
+ if (hr == DIERR_OLDDIRECTINPUTVERSION || hr ==
DIERR_BETADIRECTINPUTVERSION)
+ {
+ skip("ActionMapping requires dinput8\n");
+ return;
+ }
You probably should use win_skip here, Wine shouldn't have missing
dinput8 support.
4773
days inactive
4774
days old
2 comments
3 participants
participants (3)
-
Austin English -
Lucas Zawacki -
Vitaliy Margolen