[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.