[libvirt] [PATCH v5 0/5] qemu_process: Glib sponsored cleanup

Daniel Henrique Barboza posted 5 patches 4 years, 3 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20191220211632.700239-1-danielhb413@gmail.com
src/qemu/qemu_domain.h  |   1 +
src/qemu/qemu_process.c | 445 +++++++++++++++-------------------------
2 files changed, 166 insertions(+), 280 deletions(-)
[libvirt] [PATCH v5 0/5] qemu_process: Glib sponsored cleanup
Posted by Daniel Henrique Barboza 4 years, 3 months ago
changes from previous version 4 [1]:
- patch 1:
  * moved 'dev' inside the for loop in qemuProcessHandleBlockThreshold()
  * moved 'id' inside the 'for' loop in qemuProcessLookupPTYs()
  * moved 'path' inside both conditionals in qemuProcessBuildDestroyMemoryPaths()
  * set 'template=NULL' in case g_mkdtemp() succeeds in qemuProcessQMPInit()
- patch 2: new patch. add g_autoptr() support for
  qemuDomainLogContextPtr
- patch 3:
  * using g_autoptr() on feasible virCommandPtr and
  * qemuDomainLogContextPtr

[1] https://www.redhat.com/archives/libvir-list/2019-December/msg01311.html

Daniel Henrique Barboza (5):
  qemu_process.c: use g_autofree
  qemu_domain.h: add G_DEFINE_AUTOPTR_CLEANUP_FUNC for
    qemuDomainLogContext
  qemu_process.c: use g_autoptr()
  qemu_process.c: remove cleanup labels after g_auto*() changes
  qemu_process.c: remove 'cleanup' label from
    qemuProcessCreatePretendCmd()

 src/qemu/qemu_domain.h  |   1 +
 src/qemu/qemu_process.c | 445 +++++++++++++++-------------------------
 2 files changed, 166 insertions(+), 280 deletions(-)

-- 
2.24.1


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