https://bugs.winehq.org/show_bug.cgi?id=51490
Bug ID: 51490 Summary: dinput:device has a random failure in overlapped_format_tests() on Windows 8.1 Product: Wine Version: unspecified Hardware: x86-64 OS: Windows Status: NEW Severity: normal Priority: P2 Component: directx-dinput Assignee: wine-bugs@winehq.org Reporter: fgouget@codeweavers.com
dinput:device has a random failure on the w8 VM:
https://test.winehq.org/data/patterns.html#dinput:device
device.c:374: Test failed: keydown for D did not register
It sort of looks like a race condition though the pump_messages() call should take care of that.
The failing test (and all of overlapped_format_tests()) was added by the following commit.
commit d10887b8f56792ebcca717ccc28a289f7bcaf107 Author: Arkadiusz Hiler ahiler@codeweavers.com AuthorDate: Thu Jul 8 13:03:21 2021 +0300
dinput: Don't add 'default value 0' entries to the data transform.
This helps when dealing with the quirky c_dfDIJoystick2 format, which defines all the sliders as overlapping, i.e. rglSlider[2], rglVSlider[2], rglASlider[2] and rglFSlider[2] all share the same offset.
With the Linux backends, if the device exposes enough axes (e.g. some HOTAS) we end up using rglSlider.
Let's make sure we don't unnecessarily override the values there.
Signed-off-by: Arkadiusz Hiler ahiler@codeweavers.com Signed-off-by: Alexandre Julliard julliard@winehq.org