[PATCH 0/5] trivial qemuProcessQMPPtr related cleanups

Daniel Henrique Barboza posted 5 patches 3 years, 9 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200717211556.1024748-1-danielhb413@gmail.com
src/qemu/qemu_capabilities.c |  3 +--
src/qemu/qemu_driver.c       | 41 ++++++++++++++----------------------
src/qemu/qemu_process.c      | 13 ++++--------
src/qemu/qemu_process.h      |  1 +
4 files changed, 22 insertions(+), 36 deletions(-)
[PATCH 0/5] trivial qemuProcessQMPPtr related cleanups
Posted by Daniel Henrique Barboza 3 years, 9 months ago
Hi,

These are some simple cleanups related to the qemuProcessQMPPtr
pointer I ended up making while trying to understand how and when
qemuProcessQMPFree() is called.

Daniel Henrique Barboza (5):
  qemu_process.h: register AUTOPTR_CLEANUP_FUNC for qemuProcessQMPPtr
  qemu_process.c: modernize qemuProcessQMPNew()
  qemu_driver.c: modernize qemuConnectCPUModelBaseline()
  qemu_driver.c: modernize qemuConnectCPUModelComparison()
  qemu_capabilities.c: use g_autoptr() in virQEMUCapsInitQMPSingle()

 src/qemu/qemu_capabilities.c |  3 +--
 src/qemu/qemu_driver.c       | 41 ++++++++++++++----------------------
 src/qemu/qemu_process.c      | 13 ++++--------
 src/qemu/qemu_process.h      |  1 +
 4 files changed, 22 insertions(+), 36 deletions(-)

-- 
2.26.2

Re: [PATCH 0/5] trivial qemuProcessQMPPtr related cleanups
Posted by Jiri Denemark 3 years, 9 months ago
On Fri, Jul 17, 2020 at 18:15:51 -0300, Daniel Henrique Barboza wrote:
> Hi,
> 
> These are some simple cleanups related to the qemuProcessQMPPtr
> pointer I ended up making while trying to understand how and when
> qemuProcessQMPFree() is called.
> 
> Daniel Henrique Barboza (5):
>   qemu_process.h: register AUTOPTR_CLEANUP_FUNC for qemuProcessQMPPtr
>   qemu_process.c: modernize qemuProcessQMPNew()
>   qemu_driver.c: modernize qemuConnectCPUModelBaseline()
>   qemu_driver.c: modernize qemuConnectCPUModelComparison()
>   qemu_capabilities.c: use g_autoptr() in virQEMUCapsInitQMPSingle()
> 
>  src/qemu/qemu_capabilities.c |  3 +--
>  src/qemu/qemu_driver.c       | 41 ++++++++++++++----------------------
>  src/qemu/qemu_process.c      | 13 ++++--------
>  src/qemu/qemu_process.h      |  1 +
>  4 files changed, 22 insertions(+), 36 deletions(-)

Reviewed-by: Jiri Denemark <jdenemar@redhat.com>

And pushed.