We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

[patch] RFC: test: Add a unit test for input device classification heuristics

test-Add-a-unit-test-for-input-device-classification-heur.patch (text/plain), 16.99 KB, created by Simon McVittie on 2020-11-03 20:18:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Simon McVittie

Created: 2020-11-03 20:18:08 UTC

Size: 16.99 KB

patch

obsolete

>From: Simon McVittie <smcv@collabora.com>
>Date: Tue, 3 Nov 2020 17:03:09 +0000
>Subject: test: Add a unit test for input device classification heuristics
>
>This uses pre-recorded evdev capabilities, so that we can check for
>regressions without the devices having to be physically present.
>
>Signed-off-by: Simon McVittie <smcv@collabora.com>
>---
> test/Makefile.in |   4 +
> test/testevdev.c | 515 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 519 insertions(+)
> create mode 100644 test/testevdev.c
>
>diff --git a/test/Makefile.in b/test/Makefile.in
>index 9a62156..f2184a8 100644
>--- a/test/Makefile.in
>+++ b/test/Makefile.in
>@@ -24,6 +24,7 @@ TARGETS = \
> 	testdrawchessboard$(EXE) \
> 	testdropfile$(EXE) \
> 	testerror$(EXE) \
>+	testevdev$(EXE) \
> 	testfile$(EXE) \
> 	testfilesystem$(EXE) \
> 	testgamecontroller$(EXE) \
>@@ -146,6 +147,9 @@ testdropfile$(EXE): $(srcdir)/testdropfile.c
> testerror$(EXE): $(srcdir)/testerror.c
> 	$(CC) -o $@ $^ $(CFLAGS) $(LIBS)
> 
>+testevdev$(EXE): $(srcdir)/testevdev.c
>+	$(CC) -o $@ $^ $(CFLAGS) $(LIBS)
>+
> testfile$(EXE): $(srcdir)/testfile.c
> 	$(CC) -o $@ $^ $(CFLAGS) $(LIBS)
> 
>diff --git a/test/testevdev.c b/test/testevdev.c
>new file mode 100644
>index 0000000..f86a7a5
>--- /dev/null
>+++ b/test/testevdev.c
>@@ -0,0 +1,515 @@
>+/*
>+  Copyright (C) 1997-2020 Sam Lantinga <slouken@libsdl.org>
>+  Copyright (C) 2020 Collabora Ltd.
>+
>+  This software is provided 'as-is', without any express or implied
>+  warranty.  In no event will the authors be held liable for any damages
>+  arising from the use of this software.
>+
>+  Permission is granted to anyone to use this software for any purpose,
>+  including commercial applications, and to alter it and redistribute it
>+  freely.
>+*/
>+
>+#include "../src/SDL_internal.h"
>+
>+#include <stdio.h>
>+#include <string.h>
>+
>+static int run_test(void);
>+
>+#if HAVE_LIBUDEV_H || defined(SDL_JOYSTICK_LINUX)
>+
>+#include <stdint.h>
>+
>+#include "SDL_stdinc.h"
>+#include "SDL_endian.h"
>+#include "../src/core/linux/SDL_evdev_capabilities.h"
>+#include "../src/core/linux/SDL_evdev_capabilities.c"
>+
>+static const struct
>+{
>+    int code;
>+    const char *name;
>+} device_classes[] =
>+{
>+#define CLS(x) \
>+    { SDL_UDEV_DEVICE_ ## x, #x }
>+    CLS(MOUSE),
>+    CLS(KEYBOARD),
>+    CLS(JOYSTICK),
>+    CLS(SOUND),
>+    CLS(TOUCHSCREEN),
>+    CLS(ACCELEROMETER),
>+#undef CLS
>+    { 0, NULL }
>+};
>+
>+typedef struct
>+{
>+  const char *name;
>+  uint8_t ev[(EV_MAX + 1) / 8];
>+  uint8_t keys[(KEY_MAX + 1) / 8];
>+  uint8_t abs[(ABS_MAX + 1) / 8];
>+  uint8_t rel[(REL_MAX + 1) / 8];
>+  uint8_t ff[(FF_MAX + 1) / 8];
>+  uint8_t props[INPUT_PROP_MAX / 8];
>+  int expected;
>+} GuessTest;
>+
>+/*
>+ * Test-cases for guessing a device type from its capabilities.
>+ *
>+ * The bytes in ev, etc. are in little-endian byte order.
>+ * Trailing zeroes can be omitted.
>+ */
>+#define ZEROx4 0, 0, 0, 0
>+#define ZEROx8 ZEROx4, ZEROx4
>+#define FFx4 0xff, 0xff, 0xff, 0xff
>+#define FFx8 FFx4, FFx4
>+
>+/* Test-cases derived from real devices or from Linux kernel source */
>+static const GuessTest guess_tests[] =
>+{
>+    {
>+      .name = "Xbox 360 wired USB controller, 0003:0045e:028e",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS, FF */
>+      .ev = { 0x0b, 0x00, 0x20 },
>+      /* X, Y, Z, RX, RY, RZ, HAT0X, HAT0Y */
>+      .abs = { 0x3f, 0x00, 0x03 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A, B, X, Y, TL, TR, SELECT, START, MODE, THUMBL, THUMBR */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0xdb, 0x7c,
>+      },
>+    },
>+    {
>+      .name = "DualShock 4, 0003:054c:09cc - gamepad",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS, MSC, FF */
>+      .ev = { 0x1b, 0x00, 0x20 },
>+      /* X, Y, Z, RX, RY, RZ, HAT0X, HAT0Y */
>+      .abs = { 0x3f, 0x00, 0x03 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A, B, X, Y, TL, TR, TL2, TR2, SELECT, START, MODE,
>+           * THUMBL, THUMBR */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0xdb, 0x7f,
>+      },
>+    },
>+    {
>+      .name = "DualShock 4, 0003:054c:09cc - touchpad",
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* X, Y, multitouch */
>+      .abs = { 0x03, 0x00, 0x00, 0x00, 0x00, 0x80, 0x60, 0x02 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* Left mouse button */
>+          /* 0x100 */ 0x00, 0x00, 0x01, 0x00, ZEROx4,
>+          /* BTN_TOOL_FINGER and some multitouch stuff */
>+          /* 0x140 */ 0x20, 0x24, 0x00, 0x00
>+      },
>+      /* POINTER, BUTTONPAD */
>+      .props = { 0x05 },
>+    },
>+    {
>+      .name = "DualShock 4, 0003:054c:09cc - accelerometer",
>+      .expected = SDL_UDEV_DEVICE_ACCELEROMETER,
>+      /* SYN, ABS, MSC */
>+      .ev = { 0x19 },
>+      /* X, Y, Z, RX, RY, RZ */
>+      .abs = { 0x3f },
>+      /* ACCELEROMETER */
>+      .props = { 0x40 },
>+    },
>+    {
>+      .name = "DualShock 3, 0003:054x:0268 - gamepad",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS, MSC, FF */
>+      .ev = { 0x1b, 0x00, 0x20 },
>+      /* X, Y, Z, RX, RY, RZ */
>+      .abs = { 0x3f },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A, B, X, Y, TL, TR, TL2, TR2, SELECT, START, MODE,
>+           * THUMBL, THUMBR */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0xdb, 0x7f,
>+          /* 0x140 */ ZEROx8,
>+          /* 0x180 */ ZEROx8,
>+          /* 0x1c0 */ ZEROx8,
>+          /* Digital dpad */
>+          /* 0x200 */ ZEROx4, 0x0f, 0x00, 0x00, 0x00,
>+      },
>+    },
>+    {
>+      .name = "DualShock 3, 0003:054x:0268 - accelerometer",
>+      .expected = SDL_UDEV_DEVICE_ACCELEROMETER,
>+      /* SYN, ABS */
>+      .ev = { 0x09 },
>+      /* X, Y, Z */
>+      .abs = { 0x07 },
>+      /* ACCELEROMETER */
>+      .props = { 0x40 },
>+    },
>+    {
>+      .name = "Steam Controller, 0003:28de:1142 - gamepad",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* X, Y, RX, RY, HAT0X, HAT0Y, HAT2X, HAT2Y */
>+      .abs = { 0x1b, 0x00, 0x33 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A, B, X, Y, TL, TR, TL2, TR2, SELECT, START, MODE,
>+           * THUMBL, THUMBR, joystick THUMB, joystick THUMB2  */
>+          /* 0x100 */ ZEROx4, 0x06, 0x00, 0xdb, 0x7f,
>+          /* GEAR_DOWN, GEAR_UP */
>+          /* 0x140 */ 0x00, 0x00, 0x03, 0x00, ZEROx4,
>+          /* 0x180 */ ZEROx8,
>+          /* 0x1c0 */ ZEROx8,
>+          /* Digital dpad */
>+          /* 0x200 */ ZEROx4, 0x0f, 0x00, 0x00, 0x00,
>+      },
>+    },
>+    {
>+      /* Present to support lizard mode, even if no Steam Controller
>+       * is connected */
>+      .name = "Steam Controller, 0003:28de:1142 - dongle",
>+      .expected = (SDL_UDEV_DEVICE_KEYBOARD | SDL_UDEV_DEVICE_MOUSE),
>+      /* SYN, KEY, REL, MSC, LED, REP */
>+      .ev = { 0x17, 0x00, 0x12 },
>+      /* X, Y, mouse wheel, high-res mouse wheel */
>+      .rel = { 0x03, 0x09 },
>+      .keys = {
>+          /* 0x00 */ 0xfe, 0xff, 0xff, 0xff, FFx4,
>+          /* 0x40 */ 0xff, 0xff, 0xcf, 0x01, 0xdf, 0xff, 0x80, 0xe0,
>+          /* 0x80 */ ZEROx8,
>+          /* 0xc0 */ ZEROx8,
>+          /* 0x100 */ 0x00, 0x00, 0x1f, 0x00, ZEROx4,
>+      },
>+    },
>+    {
>+      .name = "Saitek ST290 Pro flight stick",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS, MSC */
>+      .ev = { 0x1b },
>+      /* X, Y, Z, RZ, HAT0X, HAT0Y */
>+      .abs = { 0x27, 0x00, 0x03 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* TRIGGER, THUMB, THUMB2, TOP, TOP2, PINKIE */
>+          /* 0x100 */ ZEROx4, 0x3f, 0x00, 0x00, 0x00,
>+      },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - buttons",
>+      /* TODO: Should this be JOYSTICK? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY */
>+      .ev = { 0x03 },
>+      .keys = {
>+          /* 0x00 */ ZEROx8,
>+          /* left, right, up down */
>+          /* 0x40 */ ZEROx4, 0x80, 0x16, 0x00, 0x00,
>+          /* 0x80 */ ZEROx8,
>+          /* 0xc0 */ ZEROx8,
>+          /* BTN_1, BTN_2, BTN_A, BTN_B, BTN_MODE */
>+          /* 0x100 */ 0x06, 0x00, 0x00, 0x00, 0x00, 0x00, 0x03, 0x10,
>+          /* 0x140 */ ZEROx8,
>+          /* next, previous */
>+          /* 0x180 */ 0x00, 0x00, 0x80, 0x10, ZEROx4,
>+      },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - accelerometer or Motion Plus",
>+      .expected = SDL_UDEV_DEVICE_ACCELEROMETER,
>+      /* SYN, ABS */
>+      .ev = { 0x09 },
>+      /* RX, RY, RZ */
>+      .abs = { 0x38 },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - IR positioning",
>+      /* TODO: Should this be JOYSTICK? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, ABS */
>+      .ev = { 0x09 },
>+      /* HAT0 to HAT3 */
>+      .abs = { 0x00, 0x1f },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - Nunchuck",
>+      /* TODO: Should this be JOYSTICK? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* RX, RY, RZ, hat 0 */
>+      .abs = { 0x38, 0x03 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* C, Z */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0x24, 0x00,
>+      },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - Classic Controller",
>+      /* TODO: Should this be JOYSTICK? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* Hat 1-3 */
>+      .abs = { 0x00, 0x1c },
>+      .keys = {
>+          /* 0x00 */ ZEROx8,
>+          /* left, right, up down */
>+          /* 0x40 */ ZEROx4, 0x80, 0x16, 0x00, 0x00,
>+          /* 0x80 */ ZEROx8,
>+          /* 0xc0 */ ZEROx8,
>+          /* A, B, X, Y, MODE, TL, TL2, TR, TR2 */
>+          /* 0x100 */ ZEROx4, 0x00, 0x13, 0xdb, 0x10,
>+          /* 0x140 */ ZEROx8,
>+          /* next, previous */
>+          /* 0x180 */ 0x00, 0x00, 0x80, 0x10, ZEROx4,
>+      },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - Balance Board",
>+      /* TODO: Should this be JOYSTICK? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* Hat 0-1 */
>+      .abs = { 0x00, 0x0f },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0x01, 0x00,
>+      },
>+    },
>+    {
>+      /* Flags guessed from kernel source code */
>+      .name = "Wiimote - Wii U Pro Controller",
>+      .expected = SDL_UDEV_DEVICE_JOYSTICK,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* X, Y, RX, RY */
>+      .abs = { 0x1b },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* A, B, X, Y, TL, TR, TL2, TR2, SELECT, START, MODE,
>+           * THUMBL, THUMBR */
>+          /* 0x100 */ ZEROx4, 0x00, 0x00, 0xdb, 0x7f,
>+          /* 0x140 */ ZEROx8,
>+          /* 0x180 */ ZEROx8,
>+          /* 0x1c0 */ ZEROx8,
>+          /* Digital dpad */
>+          /* 0x200 */ ZEROx4, 0x0f, 0x00, 0x00, 0x00,
>+      },
>+    },
>+    {
>+      .name = "Thinkpad trackpad",
>+      /* Should this be MOUSE? */
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, ABS */
>+      .ev = { 0x0b },
>+      /* X, Y, pressure, multitouch */
>+      .abs = { 0x03, 0x00, 0x00, 0x01, 0x00, 0x80, 0xf3, 0x06 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* Left mouse button */
>+          /* 0x100 */ 0x00, 0x00, 0x01, 0x00, ZEROx4,
>+          /* BTN_TOOL_FINGER and some multitouch gestures */
>+          /* 0x140 */ 0x20, 0xe5
>+      },
>+      /* POINTER, BUTTONPAD */
>+      .props = { 0x05 },
>+    },
>+    {
>+      .name = "Thinkpad TrackPoint",
>+      .expected = SDL_UDEV_DEVICE_MOUSE,
>+      /* SYN, KEY, REL */
>+      .ev = { 0x07 },
>+      /* X, Y */
>+      .rel = { 0x03 },
>+      .keys = {
>+          /* 0x00-0xff */ ZEROx8, ZEROx8, ZEROx8, ZEROx8,
>+          /* Left, middle, right mouse buttons */
>+          /* 0x100 */ 0x00, 0x00, 0x07,
>+      },
>+      /* POINTER, POITNING_STICK */
>+      .props = { 0x21 },
>+    },
>+    {
>+      .name = "Thinkpad ACPI buttons",
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, KEY, MSC, SW */
>+      .ev = { 0x33 },
>+      .keys = {
>+          /* 0x00 */ ZEROx8,
>+          /* 0x40 */ ZEROx4, 0x00, 0x00, 0x0e, 0x01,
>+          /* 0x80 */ 0x00, 0x50, 0x11, 0x51, 0x00, 0x28, 0x00, 0xc0,
>+          /* 0xc0 */ 0x04, 0x20, 0x10, 0x02, 0x1b, 0x70, 0x01, 0x00,
>+          /* 0x100 */ ZEROx8,
>+          /* 0x140 */ ZEROx4, 0x00, 0x00, 0x50, 0x00,
>+          /* 0x180 */ ZEROx8,
>+          /* 0x1c0 */ 0x00, 0x00, 0x04, 0x18, ZEROx4,
>+          /* 0x200 */ ZEROx8,
>+          /* 0x240 */ 0x40, 0x00, 0x01, 0x00, ZEROx4,
>+      },
>+    },
>+    {
>+      .name = "PC speaker",
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+      /* SYN, SND */
>+      .ev = { 0x01, 0x00, 0x04 },
>+    },
>+    {
>+      .name = "Thinkpad keyboard",
>+      .expected = SDL_UDEV_DEVICE_KEYBOARD,
>+      /* SYN, KEY, MSC, LED, REP */
>+      .ev = { 0x13, 0x00, 0x12 },
>+      .keys = {
>+          /* 0x00 */ 0xfe, 0xff, 0xff, 0xff, FFx4,
>+          /* 0x40 */ 0xff, 0xff, 0xef, 0xff, 0xdf, 0xff, 0xff, 0xfe,
>+          /* 0x80 */ 0x01, 0xd0, 0x00, 0xf8, 0x78, 0x30, 0x80, 0x03,
>+          /* 0xc0 */ 0x00, 0x00, 0x00, 0x02, 0x04, 0x00, 0x00, 0x00,
>+      },
>+    },
>+    {
>+      .name = "Thinkpad USB keyboard, 0003:17ef:6009 - keyboard",
>+      .expected = SDL_UDEV_DEVICE_KEYBOARD,
>+      /* SYN, KEY, MSC, LED, REP */
>+      .ev = { 0x13, 0x00, 0x12 },
>+      .keys = {
>+          /* 0x00 */ 0xfe, 0xff, 0xff, 0xff, FFx4,
>+          /* 0x40 */ 0xff, 0xff, 0xef, 0xff, 0xdf, 0xff, 0xbe, 0xfe,
>+          /* 0x80 */ 0xff, 0x57, 0x40, 0xc1, 0x7a, 0x20, 0x9f, 0xff,
>+          /* 0xc0 */ 0x07, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x00,
>+      },
>+    },
>+    {
>+      .name = "Thinkpad USB keyboard, 0003:17ef:6009 - Trackpoint",
>+      .expected = SDL_UDEV_DEVICE_MOUSE,
>+      /* SYN, KEY, REL, MSC, LED */
>+      .ev = { 0x17, 0x00, 0x02 },
>+      /* X, Y */
>+      .rel = { 0x03 },
>+      .keys = {
>+          /* 0x00 */ ZEROx8,
>+          /* 0x40 */ ZEROx4, 0x00, 0x00, 0x1e, 0x00,
>+          /* 0x80 */ 0x00, 0xcc, 0x11, 0x01, 0x78, 0x40, 0x00, 0xc0,
>+          /* 0xc0 */ 0x00, 0x20, 0x10, 0x00, 0x0b, 0x50, 0x00, 0x00,
>+          /* 0x100 */ 0x00, 0x00, 0x87, 0x68, ZEROx4,
>+          /* 0x140 */ ZEROx4, 0x00, 0x00, 0x10, 0x00,
>+          /* 0x180 */ ZEROx4, 0x00, 0x00, 0x40, 0x00,
>+      },
>+    },
>+    {
>+      .name = "No information",
>+      .expected = SDL_UDEV_DEVICE_UNKNOWN,
>+    }
>+};
>+
>+#if ULONG_MAX == 0xFFFFFFFFUL
>+#   define SwapLongLE(X) SDL_SwapLE32(X)
>+#else
>+    /* assume 64-bit */
>+#   define SwapLongLE(X) SDL_SwapLE64(X)
>+#endif
>+
>+static int
>+run_test(void)
>+{
>+    int success = 1;
>+    size_t i;
>+
>+    for (i = 0; i < SDL_arraysize(guess_tests); i++) {
>+        const GuessTest *t = &guess_tests[i];
>+        size_t j;
>+        int actual;
>+        struct {
>+            unsigned long ev[NBITS(EV_MAX)];
>+            unsigned long abs[NBITS(ABS_MAX)];
>+            unsigned long keys[NBITS(KEY_MAX)];
>+            unsigned long rel[NBITS(REL_MAX)];
>+        } caps = {};
>+
>+        printf("%s...\n", t->name);
>+
>+        memset(&caps, '\0', sizeof(caps));
>+        memcpy(caps.ev, t->ev, sizeof(t->ev));
>+        memcpy(caps.keys, t->keys, sizeof(t->keys));
>+        memcpy(caps.abs, t->abs, sizeof(t->abs));
>+        memcpy(caps.rel, t->rel, sizeof(t->rel));
>+
>+        for (j = 0; j < SDL_arraysize(caps.ev); j++) {
>+            caps.ev[j] = SwapLongLE(caps.ev[j]);
>+        }
>+
>+        for (j = 0; j < SDL_arraysize(caps.keys); j++) {
>+            caps.keys[j] = SwapLongLE(caps.keys[j]);
>+        }
>+
>+        for (j = 0; j < SDL_arraysize(caps.abs); j++) {
>+            caps.abs[j] = SwapLongLE(caps.abs[j]);
>+        }
>+
>+        for (j = 0; j < SDL_arraysize(caps.rel); j++) {
>+            caps.rel[j] = SwapLongLE(caps.rel[j]);
>+        }
>+
>+        actual = SDL_EVDEV_GuessDeviceClass(caps.ev, caps.abs, caps.keys,
>+                                            caps.rel);
>+
>+        if (actual == t->expected) {
>+            printf("\tOK\n");
>+        }
>+        else {
>+            printf("\tExpected 0x%08x\n", t->expected);
>+
>+            for (j = 0; device_classes[j].code != 0; j++) {
>+                if (t->expected & device_classes[j].code) {
>+                    printf("\t\t%s\n", device_classes[j].name);
>+                }
>+            }
>+
>+            printf("\tGot      0x%08x\n", actual);
>+
>+            for (j = 0; device_classes[j].code != 0; j++) {
>+                if (actual & device_classes[j].code) {
>+                    printf("\t\t%s\n", device_classes[j].name);
>+                }
>+            }
>+
>+            success = 0;
>+        }
>+    }
>+
>+    return success;
>+}
>+
>+#else /* !(HAVE_LIBUDEV_H || defined(SDL_JOYSTICK_LINUX)) */
>+
>+static int
>+run_test(void)
>+{
>+    printf("SDL compiled without evdev capability check.\n");
>+    return 0;
>+}
>+
>+#endif
>+
>+int
>+main(int argc, char *argv[])
>+{
>+    return run_test() ? 0 : 1;
>+}

Actions: View | Diff

Attachments on bug 5337: 4492 | 4493 | 4494 | 4495 | 4496 | 4497 | 4498 | 4499 | 4500 | 4501 | 4502 | 4503 | 4504 | 4508

We are currently migrating Bugzilla to GitHub issues.Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.