[Spice-devel] [spice-gtk v1] Revert "win-usb-dev: report error to g_udev_client_new()"

Victor Toso victortoso at redhat.com
Wed Nov 16 17:59:58 UTC 2016


From: Victor Toso <me at victortoso.com>

This reverts commit eba83a7056d4213b60466f692a03e515115b0826.

This is breaking build in linux as it relies on the actual GUdevClient
API. Let's simply revert and find a different way to get the error.

Signed-off-by: Victor Toso <victortoso at redhat.com>
---
 src/usb-device-manager.c | 5 ++---
 src/win-usb-dev.c        | 4 ++--
 src/win-usb-dev.h        | 2 +-
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
index efb7454..6d10daa 100644
--- a/src/usb-device-manager.c
+++ b/src/usb-device-manager.c
@@ -320,10 +320,9 @@ static gboolean spice_usb_device_manager_initable_init(GInitable  *initable,
 
     /* Start listening for usb devices plug / unplug */
 #ifdef USE_GUDEV
-    priv->udev = g_udev_client_new(subsystems, err);
+    priv->udev = g_udev_client_new(subsystems);
     if (priv->udev == NULL) {
-        const gchar *msg = (err != NULL && *err != NULL) ? (*err)->message : "";
-        g_warning("Error initializing GUdevClient - %s", msg);
+        g_warning("Error initializing GUdevClient");
         return FALSE;
     }
     g_signal_connect(G_OBJECT(priv->udev), "uevent",
diff --git a/src/win-usb-dev.c b/src/win-usb-dev.c
index defa71e..ec3dd91 100644
--- a/src/win-usb-dev.c
+++ b/src/win-usb-dev.c
@@ -106,12 +106,12 @@ GQuark g_udev_client_error_quark(void)
     return g_quark_from_static_string("win-gudev-client-error-quark");
 }
 
-GUdevClient *g_udev_client_new(const gchar* const *subsystems, GError **error)
+GUdevClient *g_udev_client_new(const gchar* const *subsystems)
 {
     if (singleton != NULL)
         return g_object_ref(singleton);
 
-    singleton = g_initable_new(G_UDEV_TYPE_CLIENT, NULL, error, NULL);
+    singleton = g_initable_new(G_UDEV_TYPE_CLIENT, NULL, NULL, NULL);
     return singleton;
 }
 
diff --git a/src/win-usb-dev.h b/src/win-usb-dev.h
index a9562ab..b5c4fce 100644
--- a/src/win-usb-dev.h
+++ b/src/win-usb-dev.h
@@ -79,7 +79,7 @@ struct _GUdevClientClass
 };
 
 GType g_udev_client_get_type(void) G_GNUC_CONST;
-GUdevClient *g_udev_client_new(const gchar* const *subsystems, GError **error);
+GUdevClient *g_udev_client_new(const gchar* const *subsystems);
 GList *g_udev_client_query_by_subsystem(GUdevClient *client, const gchar *subsystem);
 
 GType g_udev_device_get_type(void) G_GNUC_CONST;
-- 
2.9.3



More information about the Spice-devel mailing list