[libvirt PATCH 0/6] tests: qemu: use more g_auto

Ján Tomko posted 6 patches 2 weeks ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1595979782.git.jtomko@redhat.com
tests/qemuagenttest.c                 | 16 +++----
tests/qemublocktest.c                 |  2 +-
tests/qemucapabilitiestest.c          | 41 ++++++-----------
tests/qemucaps2xmltest.c              | 39 ++++++----------
tests/qemudomainsnapshotxml2xmltest.c | 29 +++++-------
tests/qemuhotplugtest.c               | 52 +++++++---------------
tests/qemumemlocktest.c               | 26 +++--------
tests/qemumigparamstest.c             | 62 +++++++++-----------------
tests/qemumonitortestutils.c          | 64 ++++++++++-----------------
tests/qemusecuritymock.c              | 12 ++---
tests/qemuxml2argvtest.c              | 27 ++++-------
tests/qemuxml2xmltest.c               |  9 ++--
tests/testutilsqemu.c                 | 11 ++---
tests/utiltest.c                      | 15 +------
14 files changed, 133 insertions(+), 272 deletions(-)

[libvirt PATCH 0/6] tests: qemu: use more g_auto

Posted by Ján Tomko 2 weeks ago
Ján Tomko (6):
  tests: qemu: reduce scope of some variables
  tests: qemucapsxml2xmltest: split variable declaration
  tests: qemu: use g_autofree where possible
  tests: qemu: use g_autoptr where possible
  tests: qemu: use VIR_AUTOSTRINGLIST where possible
  tests: qemu: remove unnecessary labels

 tests/qemuagenttest.c                 | 16 +++----
 tests/qemublocktest.c                 |  2 +-
 tests/qemucapabilitiestest.c          | 41 ++++++-----------
 tests/qemucaps2xmltest.c              | 39 ++++++----------
 tests/qemudomainsnapshotxml2xmltest.c | 29 +++++-------
 tests/qemuhotplugtest.c               | 52 +++++++---------------
 tests/qemumemlocktest.c               | 26 +++--------
 tests/qemumigparamstest.c             | 62 +++++++++-----------------
 tests/qemumonitortestutils.c          | 64 ++++++++++-----------------
 tests/qemusecuritymock.c              | 12 ++---
 tests/qemuxml2argvtest.c              | 27 ++++-------
 tests/qemuxml2xmltest.c               |  9 ++--
 tests/testutilsqemu.c                 | 11 ++---
 tests/utiltest.c                      | 15 +------
 14 files changed, 133 insertions(+), 272 deletions(-)

-- 
2.26.2

Re: [libvirt PATCH 0/6] tests: qemu: use more g_auto

Posted by Laine Stump 2 weeks ago
On 7/28/20 7:43 PM, Ján Tomko wrote:
> Ján Tomko (6):
>    tests: qemu: reduce scope of some variables
>    tests: qemucapsxml2xmltest: split variable declaration
>    tests: qemu: use g_autofree where possible
>    tests: qemu: use g_autoptr where possible
>    tests: qemu: use VIR_AUTOSTRINGLIST where possible
>    tests: qemu: remove unnecessary labels
>
>   tests/qemuagenttest.c                 | 16 +++----
>   tests/qemublocktest.c                 |  2 +-
>   tests/qemucapabilitiestest.c          | 41 ++++++-----------
>   tests/qemucaps2xmltest.c              | 39 ++++++----------
>   tests/qemudomainsnapshotxml2xmltest.c | 29 +++++-------
>   tests/qemuhotplugtest.c               | 52 +++++++---------------
>   tests/qemumemlocktest.c               | 26 +++--------
>   tests/qemumigparamstest.c             | 62 +++++++++-----------------
>   tests/qemumonitortestutils.c          | 64 ++++++++++-----------------
>   tests/qemusecuritymock.c              | 12 ++---
>   tests/qemuxml2argvtest.c              | 27 ++++-------
>   tests/qemuxml2xmltest.c               |  9 ++--
>   tests/testutilsqemu.c                 | 11 ++---
>   tests/utiltest.c                      | 15 +------
>   14 files changed, 133 insertions(+), 272 deletions(-)
>

Series


Reviewed-by: Laine Stump <laine@redhat.com>