[mutter/wip/carlosg/udev-client-warnings] backends/native: Shuffle udev client initialization in MetaSeatNative



commit d550d16e8883caaef38fe9430b075a67a8467ced
Author: Carlos Garnacho <carlosg gnome org>
Date:   Wed Feb 26 11:38:39 2020 +0100

    backends/native: Shuffle udev client initialization in MetaSeatNative
    
    This may be used indirectly before creation as we dispatch libinput events
    right after creation (to let input devices be known), so those device
    additions would trigger the touch-mode checks.
    
    Creating it in advance results in checks being correctly performed, although
    redundantly.
    
    Spotted by Bastien Nocera.
    
    Closes: https://gitlab.gnome.org/GNOME/mutter/issues/1067

 src/backends/native/meta-seat-native.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
---
diff --git a/src/backends/native/meta-seat-native.c b/src/backends/native/meta-seat-native.c
index f94f43c96..fbf3c8581 100644
--- a/src/backends/native/meta-seat-native.c
+++ b/src/backends/native/meta-seat-native.c
@@ -2503,6 +2503,8 @@ meta_seat_native_constructed (GObject *object)
   _clutter_input_device_set_stage (device, stage);
   seat->core_keyboard = device;
 
+  seat->udev_client = g_udev_client_new ((const gchar *[]) { "input", NULL });
+
   udev = udev_new ();
   if (G_UNLIKELY (udev == NULL))
     {
@@ -2548,10 +2550,8 @@ meta_seat_native_constructed (GObject *object)
         xkb_keymap_led_get_index (xkb_keymap, XKB_LED_NAME_SCROLL);
     }
 
-  seat->udev_client = g_udev_client_new ((const gchar *[]) { "input", NULL });
   seat->has_external_keyboard = has_external_keyboard (seat);
   seat->has_touchscreen = has_touchscreen (seat);
-
   update_touch_mode (seat);
 
   if (G_OBJECT_CLASS (meta_seat_native_parent_class)->constructed)


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