qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 26/27] tests: replace free_all() usage with g_auto


From: Marc-André Lureau
Subject: Re: [PATCH 26/27] tests: replace free_all() usage with g_auto
Date: Wed, 16 Mar 2022 16:53:05 +0400

Hi

On Wed, Mar 16, 2022 at 4:33 PM Markus Armbruster <armbru@redhat.com> wrote:
>
> marcandre.lureau@redhat.com writes:
>
> > From: Marc-André Lureau <marcandre.lureau@redhat.com>
> >
> > Use more idiomatic glib/auto-style code.
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> This is a bit of an improvement, but by its own, it's a rather weak
> justification for the previous patch.  Do you have more uses of
> g_autoptr in the pipe?


There are a lot of similar g_auto cleanups to be done in qemu code
base for QObject types. But I started with those tests, because they
use a pretty unorthodox free_all pattern and that shouldn't be
repeated.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]