[Spice-devel] [spice-gtk] usbutil: replace local function with libusb_strerror()
Frediano Ziglio
fziglio at redhat.com
Fri May 31 15:17:08 UTC 2019
>
> From: Victor Toso <me at victortoso.com>
>
> As libusb_strerror() does exactly what spice_usbutil_libusb_strerror()
> is doing. Notice that we can change the description language with
> libusb_setlocale() but keeping it in English by default (as it is) for
> now.
>
> Signed-off-by: Victor Toso <victortoso at redhat.com>
Acked-by: Frediano Ziglio <fziglio at redhat.com>
> ---
> src/channel-usbredir.c | 2 +-
> src/usb-device-manager.c | 8 ++++----
> src/usbutil.c | 36 ------------------------------------
> src/usbutil.h | 1 -
> src/win-usb-dev.c | 4 ++--
> 5 files changed, 7 insertions(+), 44 deletions(-)
>
> diff --git a/src/channel-usbredir.c b/src/channel-usbredir.c
> index 1910ff6..a54f67b 100644
> --- a/src/channel-usbredir.c
> +++ b/src/channel-usbredir.c
> @@ -292,7 +292,7 @@ static gboolean spice_usbredir_channel_open_device(
> if (rc != 0) {
> g_set_error(err, SPICE_CLIENT_ERROR, SPICE_CLIENT_ERROR_FAILED,
> "Could not open usb device: %s [%i]",
> - spice_usbutil_libusb_strerror(rc), rc);
> + libusb_strerror(rc), rc);
> return FALSE;
> }
>
> diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
> index bd42142..9391d6a 100644
> --- a/src/usb-device-manager.c
> +++ b/src/usb-device-manager.c
> @@ -281,7 +281,7 @@ static gboolean
> spice_usb_device_manager_initable_init(GInitable *initable,
> /* Initialize libusb */
> rc = libusb_init(&priv->context);
> if (rc < 0) {
> - const char *desc = spice_usbutil_libusb_strerror(rc);
> + const char *desc = libusb_strerror(rc);
> g_warning("Error initializing USB support: %s [%i]", desc, rc);
> g_set_error(err, SPICE_CLIENT_ERROR, SPICE_CLIENT_ERROR_FAILED,
> "Error initializing USB support: %s [%i]", desc, rc);
> @@ -308,7 +308,7 @@ static gboolean
> spice_usb_device_manager_initable_init(GInitable *initable,
> LIBUSB_HOTPLUG_MATCH_ANY, LIBUSB_HOTPLUG_MATCH_ANY,
> spice_usb_device_manager_hotplug_cb, self, &priv->hp_handle);
> if (rc < 0) {
> - const char *desc = spice_usbutil_libusb_strerror(rc);
> + const char *desc = libusb_strerror(rc);
> g_warning("Error initializing USB hotplug support: %s [%i]", desc,
> rc);
> g_set_error(err, SPICE_CLIENT_ERROR, SPICE_CLIENT_ERROR_FAILED,
> "Error initializing USB hotplug support: %s [%i]", desc,
> rc);
> @@ -730,7 +730,7 @@ static gboolean
> spice_usb_device_manager_get_device_descriptor(
>
> bus = libusb_get_bus_number(libdev);
> addr = libusb_get_device_address(libdev);
> - errstr = spice_usbutil_libusb_strerror(errcode);
> + errstr = libusb_strerror(errcode);
> g_warning("cannot get device descriptor for (%p) %d.%d -- %s(%d)",
> libdev, bus, addr, errstr, errcode);
> return FALSE;
> @@ -1068,7 +1068,7 @@ static gpointer
> spice_usb_device_manager_usb_ev_thread(gpointer user_data)
> while (g_atomic_int_get(&priv->event_thread_run)) {
> rc = libusb_handle_events(priv->context);
> if (rc && rc != LIBUSB_ERROR_INTERRUPTED) {
> - const char *desc = spice_usbutil_libusb_strerror(rc);
> + const char *desc = libusb_strerror(rc);
> g_warning("Error handling USB events: %s [%i]", desc, rc);
> break;
> }
> diff --git a/src/usbutil.c b/src/usbutil.c
> index e96ab11..5052ef3 100644
> --- a/src/usbutil.c
> +++ b/src/usbutil.c
> @@ -58,42 +58,6 @@ static GMutex usbids_load_mutex;
> static int usbids_vendor_count = 0; /* < 0: failed, 0: empty, > 0: loaded */
> static usb_vendor_info *usbids_vendor_info = NULL;
>
> -G_GNUC_INTERNAL
> -const char *spice_usbutil_libusb_strerror(enum libusb_error error_code)
> -{
> - switch (error_code) {
> - case LIBUSB_SUCCESS:
> - return "Success";
> - case LIBUSB_ERROR_IO:
> - return "Input/output error";
> - case LIBUSB_ERROR_INVALID_PARAM:
> - return "Invalid parameter";
> - case LIBUSB_ERROR_ACCESS:
> - return "Access denied (insufficient permissions)";
> - case LIBUSB_ERROR_NO_DEVICE:
> - return "No such device (it may have been disconnected)";
> - case LIBUSB_ERROR_NOT_FOUND:
> - return "Entity not found";
> - case LIBUSB_ERROR_BUSY:
> - return "Resource busy";
> - case LIBUSB_ERROR_TIMEOUT:
> - return "Operation timed out";
> - case LIBUSB_ERROR_OVERFLOW:
> - return "Overflow";
> - case LIBUSB_ERROR_PIPE:
> - return "Pipe error";
> - case LIBUSB_ERROR_INTERRUPTED:
> - return "System call interrupted (perhaps due to signal)";
> - case LIBUSB_ERROR_NO_MEM:
> - return "Insufficient memory";
> - case LIBUSB_ERROR_NOT_SUPPORTED:
> - return "Operation not supported or unimplemented on this platform";
> - case LIBUSB_ERROR_OTHER:
> - return "Other error";
> - }
> - return "Unknown error";
> -}
> -
> #ifdef __linux__
> /* <Sigh> libusb does not allow getting the manufacturer and product strings
> without opening the device, so grab them directly from sysfs */
> diff --git a/src/usbutil.h b/src/usbutil.h
> index de5e92a..50e3949 100644
> --- a/src/usbutil.h
> +++ b/src/usbutil.h
> @@ -28,7 +28,6 @@
>
> G_BEGIN_DECLS
>
> -const char *spice_usbutil_libusb_strerror(enum libusb_error error_code);
> void spice_usb_util_get_device_strings(int bus, int address,
> int vendor_id, int product_id,
> gchar **manufacturer, gchar
> **product);
> diff --git a/src/win-usb-dev.c b/src/win-usb-dev.c
> index a4dfa78..b0274dd 100644
> --- a/src/win-usb-dev.c
> +++ b/src/win-usb-dev.c
> @@ -113,7 +113,7 @@ g_udev_client_list_devices(GUdevClient *self, GList
> **devs,
>
> rc = libusb_get_device_list(priv->ctx, &lusb_list);
> if (rc < 0) {
> - const char *errstr = spice_usbutil_libusb_strerror(rc);
> + const char *errstr = libusb_strerror(rc);
> g_warning("%s: libusb_get_device_list failed - %s", name, errstr);
> g_set_error(err, G_UDEV_CLIENT_ERROR, G_UDEV_CLIENT_LIBUSB_FAILED,
> "%s: Error getting device list from libusb: %s
> [%"G_GSSIZE_FORMAT"]",
> @@ -170,7 +170,7 @@ g_udev_client_initable_init(GInitable *initable,
> GCancellable *cancellable,
>
> rc = libusb_init(&priv->ctx);
> if (rc < 0) {
> - const char *errstr = spice_usbutil_libusb_strerror(rc);
> + const char *errstr = libusb_strerror(rc);
> g_warning("Error initializing USB support: %s [%i]", errstr, rc);
> g_set_error(err, G_UDEV_CLIENT_ERROR, G_UDEV_CLIENT_LIBUSB_FAILED,
> "Error initializing USB support: %s [%i]", errstr, rc);
More information about the Spice-devel
mailing list