Re: [PATCH v2] qapi: enable use of g_autoptr with QAPI types

[Eric Blake]

On 7/23/20 10:38 AM, Daniel P. Berrangé wrote:
> Currently QAPI generates a type and function for free'ing it:
>
>    typedef struct QCryptoBlockCreateOptions QCryptoBlockCreateOptions;
>    void qapi_free_QCryptoBlockCreateOptions(QCryptoBlockCreateOptions *obj);

> The above code example now becomes
>
>    g_autoptr(QCryptoBlockCreateOptions) opts = NULL;
>
>    opts = g_new0(QCryptoBlockCreateOptions, 1);
>
>    ....do stuff with opts...
>
> Note, if the local pointer needs to live beyond the scope holding the
> variable, then g_steal_pointer can be used. This is useful to return the
> pointer to the caller in the success codepath, while letting it be freed
> in all error codepaths.
>
>    return g_steal_pointer(&opts);
>
> The crypto/block.h header needs updating to avoid symbol clash now that
> the g_autoptr support is a standard QAPI feature.
>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---

Reviewed-by: Eric Blake <eblake@redhat.com>

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org