[Spice-devel] [spice-gtk 2/2] win-usb-dev: Use 'stable' libusb_device pointers in GUdevClient::udev_list

Yuri Benditovich yuri.benditovich at daynix.com
Wed Mar 27 08:48:45 UTC 2019


On Tue, Mar 26, 2019 at 3:11 PM Christophe Fergeau <cfergeau at redhat.com> wrote:
>
> On Mon, Mar 25, 2019 at 08:03:05PM +0200, Yuri Benditovich wrote:
> > Yes, it looks like this finally does the same thing.
>
> > From my personal point of view, this is less obvious than previous
> > code, so I would add 2 comments:
> > that we unreference the device from the new list
> > that we add reference to the device from the old list, it will be
> > dereferenced on next line when the old list freed
>
> If the part you find less clear is:
> +            libusb_unref_device(dev->data);
> +            dev->data = libusb_ref_device(found->data);
>

Yes, it is. It took me some time to understand the intention, so I suppose
some other (not too smart) readers will also need some thinking and
the comments would make the reading easier.
Again, this is my personal opinion, no objections as soon as the code
does the same thing.

> I don't really mind changing it back to do the swapping with an
> intermediate variable, even if I personally find the unref/ref to be
> more readable than the other version ;)
>
> Christophe


More information about the Spice-devel mailing list