[Bug 51490] New: dinput:device has a random failure in overlapped_format_tests() on Windows 8.1

WineHQ Bugzilla wine-bugs at winehq.org
Tue Jul 20 06:04:35 CDT 2021


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 at winehq.org
          Reporter: fgouget at 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 at 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 at codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard at winehq.org>

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.



More information about the wine-bugs mailing list