[v1] tests: use g_auto more (glib chronicles)

[libvirt PATCH 0/2] tests: use g_auto more (glib chronicles)

Ján Tomko posted 2 patches 2 years, 8 months ago

Diff against v1
Download series mbox

Test syntax-check failed

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

There is a newer version of this series

tests/qemucaps2xmltest.c | 16 ++++++----------
tests/testutilsqemu.c    | 10 +++-------
2 files changed, 9 insertions(+), 17 deletions(-)

Expand all Fold all

[libvirt PATCH 0/2] tests: use g_auto more (glib chronicles)

Posted by Ján Tomko 2 years, 8 months ago

Convert two more functions to use automatic cleanup.

Ján Tomko (2):
  tests: use g_auto in testQemuGetCaps
  tests: use g_auto in qemuTestParseCapabilitiesArch

 tests/qemucaps2xmltest.c | 16 ++++++----------
 tests/testutilsqemu.c    | 10 +++-------
 2 files changed, 9 insertions(+), 17 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH 0/2] tests: use g_auto more (glib chronicles)

Posted by Martin Kletzander 2 years, 8 months ago

On Fri, Aug 20, 2021 at 01:57:53PM +0200, Ján Tomko wrote:
>Convert two more functions to use automatic cleanup.
>
>Ján Tomko (2):
>  tests: use g_auto in testQemuGetCaps
>  tests: use g_auto in qemuTestParseCapabilitiesArch
>
> tests/qemucaps2xmltest.c | 16 ++++++----------
> tests/testutilsqemu.c    | 10 +++-------
> 2 files changed, 9 insertions(+), 17 deletions(-)
>

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

>-- 
>2.31.1
>

Re: [libvirt PATCH 0/2] tests: use g_auto more (glib chronicles)

Posted by Kristina Hanicova 2 years, 8 months ago

On Fri, Aug 20, 2021 at 2:01 PM Ján Tomko <jtomko@redhat.com> wrote:

> Convert two more functions to use automatic cleanup.
>
> Ján Tomko (2):
>   tests: use g_auto in testQemuGetCaps
>   tests: use g_auto in qemuTestParseCapabilitiesArch
>
>  tests/qemucaps2xmltest.c | 16 ++++++----------
>  tests/testutilsqemu.c    | 10 +++-------
>  2 files changed, 9 insertions(+), 17 deletions(-)
>
> --
> 2.31.1
>
>
Reviewed-by:  Kristina Hanicova <khanicov@redhat.com>

Kristina