[mutter/wip/carlosg/no-device-id: 9/10] backends/native: Don't fake device IDs




commit 20e6464087341deaa7b304e13ab9cd54318b8de2
Author: Carlos Garnacho <carlosg gnome org>
Date:   Tue Nov 17 15:09:59 2020 +0100

    backends/native: Don't fake device IDs
    
    Stop making them up, they go nowhere.
    
    Part-of: <https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1569>

 src/backends/native/meta-input-device-native.c | 14 +--------
 src/backends/native/meta-seat-native.c         | 43 --------------------------
 2 files changed, 1 insertion(+), 56 deletions(-)
---
diff --git a/src/backends/native/meta-input-device-native.c b/src/backends/native/meta-input-device-native.c
index 072c80792f..fbbe931c2c 100644
--- a/src/backends/native/meta-input-device-native.c
+++ b/src/backends/native/meta-input-device-native.c
@@ -59,18 +59,11 @@ static void stop_mousekeys_move  (MetaInputDeviceNative *device);
 static void
 meta_input_device_native_finalize (GObject *object)
 {
-  ClutterInputDevice *device = CLUTTER_INPUT_DEVICE (object);
   MetaInputDeviceNative *device_evdev = META_INPUT_DEVICE_NATIVE (object);
-  ClutterBackend *backend;
-  ClutterSeat *seat;
 
   if (device_evdev->libinput_device)
     libinput_device_unref (device_evdev->libinput_device);
 
-  backend = clutter_get_default_backend ();
-  seat = clutter_backend_get_default_seat (backend);
-  meta_seat_native_release_device_id (META_SEAT_NATIVE (seat), device);
-
   clear_slow_keys (device_evdev);
   stop_bounce_keys (device_evdev);
   stop_toggle_slowkeys (device_evdev);
@@ -1281,14 +1274,13 @@ meta_input_device_native_new (MetaSeatNative         *seat,
   MetaInputDeviceNative *device;
   ClutterInputDeviceType type;
   char *vendor, *product;
-  int device_id, n_rings = 0, n_strips = 0, n_groups = 1;
+  int n_rings = 0, n_strips = 0, n_groups = 1;
   char *node_path;
   double width, height;
 
   type = meta_input_device_native_determine_type (libinput_device);
   vendor = g_strdup_printf ("%.4x", libinput_device_get_id_vendor (libinput_device));
   product = g_strdup_printf ("%.4x", libinput_device_get_id_product (libinput_device));
-  device_id = meta_seat_native_acquire_device_id (seat);
   node_path = g_strdup_printf ("/dev/input/%s", libinput_device_get_sysname (libinput_device));
 
   if (libinput_device_has_capability (libinput_device,
@@ -1300,7 +1292,6 @@ meta_input_device_native_new (MetaSeatNative         *seat,
     }
 
   device = g_object_new (META_TYPE_INPUT_DEVICE_NATIVE,
-                         "id", device_id,
                          "name", libinput_device_get_name (libinput_device),
                          "device-type", type,
                          "device-mode", CLUTTER_INPUT_MODE_PHYSICAL,
@@ -1343,7 +1334,6 @@ meta_input_device_native_new_virtual (MetaSeatNative         *seat,
 {
   MetaInputDeviceNative *device;
   const char *name;
-  int device_id;
 
   switch (type)
     {
@@ -1361,9 +1351,7 @@ meta_input_device_native_new_virtual (MetaSeatNative         *seat,
       break;
     };
 
-  device_id = meta_seat_native_acquire_device_id (seat);
   device = g_object_new (META_TYPE_INPUT_DEVICE_NATIVE,
-                         "id", device_id,
                          "name", name,
                          "device-type", type,
                          "device-mode", mode,
diff --git a/src/backends/native/meta-seat-native.c b/src/backends/native/meta-seat-native.c
index 6c1f44990d..3df16761a1 100644
--- a/src/backends/native/meta-seat-native.c
+++ b/src/backends/native/meta-seat-native.c
@@ -3050,49 +3050,6 @@ meta_seat_native_update_xkb_state (MetaSeatNative *seat)
   meta_seat_native_sync_leds (seat);
 }
 
-gint
-meta_seat_native_acquire_device_id (MetaSeatNative *seat)
-{
-  GList *first;
-  gint next_id;
-
-  if (seat->free_device_ids == NULL)
-    {
-      gint i;
-
-      /* We ran out of free ID's, so append 10 new ones. */
-      for (i = 0; i < 10; i++)
-        seat->free_device_ids =
-          g_list_append (seat->free_device_ids,
-                         GINT_TO_POINTER (seat->device_id_next++));
-    }
-
-  first = g_list_first (seat->free_device_ids);
-  next_id = GPOINTER_TO_INT (first->data);
-  seat->free_device_ids = g_list_delete_link (seat->free_device_ids, first);
-
-  return next_id;
-}
-
-static int
-compare_ids (gconstpointer a,
-             gconstpointer b)
-{
-  return GPOINTER_TO_INT (a) - GPOINTER_TO_INT (b);
-}
-
-void
-meta_seat_native_release_device_id (MetaSeatNative     *seat,
-                                    ClutterInputDevice *device)
-{
-  gint device_id;
-
-  device_id = clutter_input_device_get_device_id (device);
-  seat->free_device_ids = g_list_insert_sorted (seat->free_device_ids,
-                                                GINT_TO_POINTER (device_id),
-                                                compare_ids);
-}
-
 /**
  * meta_seat_native_release_devices:
  *


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]