Use virURIFree less (glib chronicles)

[libvirt PATCH 0/6] Use virURIFree less (glib chronicles)

Ján Tomko posted 6 patches 1 year, 10 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1655876067.git.jtomko@redhat.com

src/esx/esx_driver.c        |  3 +--
src/libxl/libxl_migration.c | 17 ++++++-----------
src/qemu/qemu_migration.c   | 30 ++++++++++++------------------
src/vmx/vmx.c               | 13 ++++---------
src/vz/vz_driver.c          |  3 +--
src/vz/vz_sdk.c             | 31 +++++++++++--------------------
tests/viruritest.c          | 14 +++++---------
7 files changed, 40 insertions(+), 71 deletions(-)

Expand all Fold all

[libvirt PATCH 0/6] Use virURIFree less (glib chronicles)

Posted by Ján Tomko 1 year, 10 months ago

Ján Tomko (6):
  Use g_auto for virURI almost everywhere
  tests: remove pointless label in testURIParse
  vz: refactor testURIParse
  vmx: use g_autofree in virVMXParseSerial
  qemu: remove cleanup label from qemuMigrationSrcGraphicsRelocate
  libxl: remove cleanup label from libxlDomainMigrationSrcPerform

 src/esx/esx_driver.c        |  3 +--
 src/libxl/libxl_migration.c | 17 ++++++-----------
 src/qemu/qemu_migration.c   | 30 ++++++++++++------------------
 src/vmx/vmx.c               | 13 ++++---------
 src/vz/vz_driver.c          |  3 +--
 src/vz/vz_sdk.c             | 31 +++++++++++--------------------
 tests/viruritest.c          | 14 +++++---------
 7 files changed, 40 insertions(+), 71 deletions(-)

-- 
2.34.1

Re: [libvirt PATCH 0/6] Use virURIFree less (glib chronicles)

Posted by Pavel Hrdina 1 year, 10 months ago

On Wed, Jun 22, 2022 at 07:35:28AM +0200, Ján Tomko wrote:
> Ján Tomko (6):
>   Use g_auto for virURI almost everywhere
>   tests: remove pointless label in testURIParse
>   vz: refactor testURIParse
>   vmx: use g_autofree in virVMXParseSerial
>   qemu: remove cleanup label from qemuMigrationSrcGraphicsRelocate
>   libxl: remove cleanup label from libxlDomainMigrationSrcPerform

With the indentation fixed

Reviewed-by: Pavel Hrdina <phrdina@redhat.com>