dinput: Remove unusual HID joystick button count FIXMEs.

Signed-off-by: Rémi Bernon <rbernon@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

dinput: Remove unusual HID joystick button count FIXMEs.
e4766cad · Rémi Bernon · Alexandre Julliard · 5732347c · e4766cad
Commit e4766cad authored Sep 28, 2021 by Rémi Bernon Committed by Alexandre Julliard Sep 28, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 5 deletions

joystick_hid.c dlls/dinput/joystick_hid.c +1 -5

No files found.
--- a/dlls/dinput/joystick_hid.c
+++ b/dlls/dinput/joystick_hid.c
@@ -359,9 +359,7 @@ static BOOL enum_objects( struct hid_joystick *impl, const DIPROPHEADER *header,
            TRACE( "Ignoring input button %s, usage page not implemented.\n", debugstr_hid_caps( &caps ) );
        else if (caps.button->IsRange)
        {
-            if (caps.button->NotRange.Usage >= 128)
+            for (j = caps.button->Range.UsageMin; j <= caps.button->Range.UsageMax; ++j)
-                FIXME( "Ignoring input button %s, too many buttons.\n", debugstr_hid_caps( &caps ) );
-            else for (j = caps.button->Range.UsageMin; j <= caps.button->Range.UsageMax; ++j)
            {
                instance.dwOfs = button_ofs + (j - caps.button->Range.UsageMin);
                instance.dwType = DIDFT_PSHBUTTON | DIDFT_MAKEINSTANCE( button++ );
@@ -375,8 +373,6 @@ static BOOL enum_objects( struct hid_joystick *impl, const DIPROPHEADER *header,
                if (ret != DIENUM_CONTINUE) return ret;
            }
        }
-        else if (caps.button->NotRange.Usage >= 128)
-            FIXME( "Ignoring input button %s, too many buttons.\n", debugstr_hid_caps( &caps ) );
        else
        {
            instance.dwOfs = button_ofs;