Re: [PATCH v2 0/2] a few cleanups in qemu_hotplug.c

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

 



On 10/17/19 11:44 AM, Daniel Henrique Barboza wrote:
> There is not much to do regarding cleanup and g_auto* in this
> file. Here's a couple of simple fixes I've found.
> 
> 
> changes from v1:
> - added a new patch to remove unused cleanup labels
> 
> 
> v1: https://www.redhat.com/archives/libvir-list/2019-September/msg01463.html
> 
> Daniel Henrique Barboza (2):
>   qemu_hotplug.c: use g_autoptr() with virConnectPtr
>   qemu_process: remove unused cleanup labels
> 
>  src/qemu/qemu_hotplug.c | 72 +++++++++++++++++------------------------
>  1 file changed, 29 insertions(+), 43 deletions(-)
> 

I tweaked the patch subjects, to use 'qemu: hotplug:' prefix, the X.c
format isn't typically used. Patch 2 incorrectly mentioned qemu_process,
so I fixed that too

Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx>

and pushed

- Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux