[libvirt PATCH 0/4] tests: bhyve: use more g_auto

Ján Tomko posted 4 patches 3 years, 8 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1595979850.git.jtomko@redhat.com
tests/bhyveargv2xmltest.c | 47 ++++++++++++++-------------------------
tests/bhyvexml2argvtest.c | 33 ++++++++++-----------------
tests/bhyvexml2xmltest.c  |  6 ++---
3 files changed, 31 insertions(+), 55 deletions(-)
[libvirt PATCH 0/4] tests: bhyve: use more g_auto
Posted by Ján Tomko 3 years, 8 months ago
Ján Tomko (4):
  tests: bhyve: split variable declarations
  tests: bhyve: use g_autofree where possible
  tests: bhyve: use g_autoptr where possible
  tests: bhyve: remove unnecessary labels

 tests/bhyveargv2xmltest.c | 47 ++++++++++++++-------------------------
 tests/bhyvexml2argvtest.c | 33 ++++++++++-----------------
 tests/bhyvexml2xmltest.c  |  6 ++---
 3 files changed, 31 insertions(+), 55 deletions(-)

-- 
2.26.2

Re: [libvirt PATCH 0/4] tests: bhyve: use more g_auto
Posted by Laine Stump 3 years, 8 months ago
On 7/28/20 7:44 PM, Ján Tomko wrote:
> Ján Tomko (4):
>    tests: bhyve: split variable declarations
>    tests: bhyve: use g_autofree where possible
>    tests: bhyve: use g_autoptr where possible
>    tests: bhyve: remove unnecessary labels
>
>   tests/bhyveargv2xmltest.c | 47 ++++++++++++++-------------------------
>   tests/bhyvexml2argvtest.c | 33 ++++++++++-----------------
>   tests/bhyvexml2xmltest.c  |  6 ++---
>   3 files changed, 31 insertions(+), 55 deletions(-)
>

Series


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

Re: [libvirt PATCH 0/4] tests: bhyve: use more g_auto
Posted by Roman Bogorodskiy 3 years, 8 months ago
  Laine Stump wrote:

> On 7/28/20 7:44 PM, Ján Tomko wrote:
> > Ján Tomko (4):
> >    tests: bhyve: split variable declarations
> >    tests: bhyve: use g_autofree where possible
> >    tests: bhyve: use g_autoptr where possible
> >    tests: bhyve: remove unnecessary labels
> >
> >   tests/bhyveargv2xmltest.c | 47 ++++++++++++++-------------------------
> >   tests/bhyvexml2argvtest.c | 33 ++++++++++-----------------
> >   tests/bhyvexml2xmltest.c  |  6 ++---
> >   3 files changed, 31 insertions(+), 55 deletions(-)
> >
> 
> Series
> 
> 
> Reviewed-by: Laine Stump <laine@redhat.com>
> 

Reviewed-by: Roman Bogorodskiy <bogorodskiy@gmail.com>

Roman Bogorodskiy