Re: [PATCH v6 03/10] Add SpiceUsbDeviceManager parameter to device comparison functions

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

 



It doesn't seem that these new parameters are used until patch #7, unless I
missed something. So I'd re-order the patch series so that this comes right
before it's used (or even merge them together?).

Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>


On Thu, 2015-10-29 at 17:26 +0200, Dmitry Fleytman wrote:
> From: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> 
> This additional parameter is currently unused, but this is in
> preparation for the next commits.
> ---
>  src/usb-device-manager.c | 40 ++++++++++++++++++++++------------------
>  1 file changed, 22 insertions(+), 18 deletions(-)
> 
> diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
> index d261dce..4930613 100644
> --- a/src/usb-device-manager.c
> +++ b/src/usb-device-manager.c
> @@ -184,8 +184,9 @@ static guint8 spice_usb_device_get_state(SpiceUsbDevice
> *device);
>  static void  spice_usb_device_set_state(SpiceUsbDevice *device, guint8 s);
>  #endif
>  
> -static gboolean spice_usb_device_equal_libdev(SpiceUsbDevice *device,
> -                                              libusb_device *libdev);
> +static gboolean spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager
> *manager,
> +                                                      SpiceUsbDevice *device,
> +                                                      libusb_device *libdev);
>  static libusb_device *
>  spice_usb_device_manager_device_to_libdev(SpiceUsbDeviceManager *self,
>                                            SpiceUsbDevice *device);
> @@ -641,7 +642,8 @@ static void
> spice_usb_device_manager_class_init(SpiceUsbDeviceManagerClass *klas
>  
>  #ifdef USE_GUDEV
>  static gboolean spice_usb_device_manager_get_udev_bus_n_address(
> -    GUdevDevice *udev, int *bus, int *address)
> +    SpiceUsbDeviceManager *manager, GUdevDevice *udev,
> +    int *bus, int *address)
>  {
>      const gchar *bus_str, *address_str;
>  
> @@ -794,7 +796,7 @@ static void
> spice_usb_device_manager_auto_connect_cb(GObject      *gobject,
>  
>  #ifndef G_OS_WIN32 /* match functions for Linux -- match by bus.addr */
>  static gboolean
> -spice_usb_device_manager_device_match(SpiceUsbDevice *device,
> +spice_usb_device_manager_device_match(SpiceUsbDeviceManager *self,
> SpiceUsbDevice *device,
>                                        const int bus, const int address)
>  {
>      return (spice_usb_device_get_busnum(device) == bus &&
> @@ -803,7 +805,7 @@ spice_usb_device_manager_device_match(SpiceUsbDevice
> *device,
>  
>  #ifdef USE_GUDEV
>  static gboolean
> -spice_usb_device_manager_libdev_match(libusb_device *libdev,
> +spice_usb_device_manager_libdev_match(SpiceUsbDeviceManager *self,
> libusb_device *libdev,
>                                        const int bus, const int address)
>  {
>      return (libusb_get_bus_number(libdev) == bus &&
> @@ -813,7 +815,7 @@ spice_usb_device_manager_libdev_match(libusb_device
> *libdev,
>  
>  #else /* Win32 -- match functions for Windows -- match by vid:pid */
>  static gboolean
> -spice_usb_device_manager_device_match(SpiceUsbDevice *device,
> +spice_usb_device_manager_device_match(SpiceUsbDeviceManager *self,
> SpiceUsbDevice *device,
>                                        const int vid, const int pid)
>  {
>      return (spice_usb_device_get_vid(device) == vid &&
> @@ -821,7 +823,7 @@ spice_usb_device_manager_device_match(SpiceUsbDevice
> *device,
>  }
>  
>  static gboolean
> -spice_usb_device_manager_libdev_match(libusb_device *libdev,
> +spice_usb_device_manager_libdev_match(SpiceUsbDeviceManager *self,
> libusb_device *libdev,
>                                        const int vid, const int pid)
>  {
>      int vid2, pid2;
> @@ -843,7 +845,7 @@ spice_usb_device_manager_find_device(SpiceUsbDeviceManager
> *self,
>  
>      for (i = 0; i < priv->devices->len; i++) {
>          curr = g_ptr_array_index(priv->devices, i);
> -        if (spice_usb_device_manager_device_match(curr, bus, address)) {
> +        if (spice_usb_device_manager_device_match(self, curr, bus, address))
> {
>              device = curr;
>              break;
>          }
> @@ -946,7 +948,7 @@ static void
> spice_usb_device_manager_add_udev(SpiceUsbDeviceManager  *self,
>      if (!devtype || strcmp(devtype, "usb_device"))
>          return;
>  
> -    if (!spice_usb_device_manager_get_udev_bus_n_address(udev, &bus,
> &address)) {
> +    if (!spice_usb_device_manager_get_udev_bus_n_address(self, udev, &bus,
> &address)) {
>          g_warning("USB device without bus number or device address");
>          return;
>      }
> @@ -967,7 +969,7 @@ static void
> spice_usb_device_manager_add_udev(SpiceUsbDeviceManager  *self,
>          libusb_get_device_list(priv->context, &dev_list);
>  
>      for (i = 0; dev_list && dev_list[i]; i++) {
> -        if (spice_usb_device_manager_libdev_match(dev_list[i], bus, address))
> {
> +        if (spice_usb_device_manager_libdev_match(self, dev_list[i], bus,
> address)) {
>              libdev = dev_list[i];
>              break;
>          }
> @@ -988,7 +990,7 @@ static void
> spice_usb_device_manager_remove_udev(SpiceUsbDeviceManager  *self,
>  {
>      int bus, address;
>  
> -    if (!spice_usb_device_manager_get_udev_bus_n_address(udev, &bus,
> &address))
> +    if (!spice_usb_device_manager_get_udev_bus_n_address(self, udev, &bus,
> &address))
>          return;
>  
>      spice_usb_device_manager_remove_dev(self, bus, address);
> @@ -1291,7 +1293,7 @@ static SpiceUsbredirChannel
> *spice_usb_device_manager_get_channel_for_dev(
>      for (i = 0; i < priv->channels->len; i++) {
>          SpiceUsbredirChannel *channel = g_ptr_array_index(priv->channels, i);
>          libusb_device *libdev = spice_usbredir_channel_get_device(channel);
> -        if (spice_usb_device_equal_libdev(device, libdev))
> +        if (spice_usb_manager_device_equal_libdev(manager, device, libdev))
>              return channel;
>      }
>  #endif
> @@ -1863,8 +1865,9 @@ static void spice_usb_device_unref(SpiceUsbDevice
> *device)
>  
>  #ifndef G_OS_WIN32 /* Linux -- directly compare libdev */
>  static gboolean
> -spice_usb_device_equal_libdev(SpiceUsbDevice *device,
> -                              libusb_device  *libdev)
> +spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager *manager,
> +                                      SpiceUsbDevice *device,
> +                                      libusb_device  *libdev)
>  {
>      SpiceUsbDeviceInfo *info = (SpiceUsbDeviceInfo *)device;
>  
> @@ -1875,8 +1878,9 @@ spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>  }
>  #else /* Windows -- compare vid:pid of device and libdev */
>  static gboolean
> -spice_usb_device_equal_libdev(SpiceUsbDevice *device,
> -                              libusb_device  *libdev)
> +spice_usb_manager_device_equal_libdev(SpiceUsbDeviceManager *manager,
> +                                      SpiceUsbDevice *device,
> +                                      libusb_device  *libdev)
>  {
>      int vid, pid;
>  
> @@ -1885,7 +1889,7 @@ spice_usb_device_equal_libdev(SpiceUsbDevice *device,
>  
>      vid = spice_usb_device_get_vid(device);
>      pid = spice_usb_device_get_pid(device);
> -    return spice_usb_device_manager_libdev_match(libdev, vid, pid);
> +    return spice_usb_device_manager_libdev_match(manager, libdev, vid, pid);
>  }
>  #endif
>  
> @@ -1918,7 +1922,7 @@
> spice_usb_device_manager_device_to_libdev(SpiceUsbDeviceManager *self,
>          return NULL;
>  
>      for (i = 0; (d = devlist[i]) != NULL; i++) {
> -        if (spice_usb_device_equal_libdev(device, d)) {
> +        if (spice_usb_manager_device_equal_libdev(self, device, d)) {
>              libusb_ref_device(d);
>              break;
>          }
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]