[libvirt] [PATCH v4 0/5] qemu_driver: GLib powered cleanup

Michal Privoznik mprivozn at redhat.com
Thu Oct 17 12:10:08 UTC 2019


On 10/16/19 10:54 PM, Daniel Henrique Barboza wrote:
> changes from v3:
> - only use the new Glib macros in all patches
> - different patch split, as suggested by Jano in v3 review
> - patch 5 (new): g_strdup_printf conversion
> 
> changes from v2:
> - rebased with newer master (67e72053c1)
> - added an extra patch to convert the existing VIR_AUTO* macros
> to g_auto* ones, all at once, to avoid the case where a method
> will have both VIR_AUTO* and g_auto* macros at the same time.
> 
> changes from v1:
> - addressed review concerns made by Erik
> - added more cleanups, as suggested by Erik
> 
> v3: https://www.redhat.com/archives/libvir-list/2019-October/msg00918.html
> v2: https://www.redhat.com/archives/libvir-list/2019-September/msg01452.html
> v1: https://www.redhat.com/archives/libvir-list/2019-September/msg00719.html
> 
> Daniel Henrique Barboza (5):
>    qemu_driver: use g_auto* in some functions
>    qemu_driver: use g_autoptr() when possible
>    qemu_driver: use g_autofree when possible
>    qemu_driver: remove unused 'cleanup' labels after g_auto*() changes
>    qemu_driver: use g_strdup_printf
> 
>   src/qemu/qemu_driver.c | 721 ++++++++++++++---------------------------
>   1 file changed, 242 insertions(+), 479 deletions(-)
> 


I'll replace qemu_driver with qemu_driver.c in subjects because you're 
touching qemu_driver.c only.

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

And pushed.

Michal




More information about the libvir-list mailing list