Paolo Bonzini <[email protected]> writes: > On 13/09/21 07:23, Markus Armbruster wrote: >> Paolo Bonzini <[email protected]> writes: >> >>> Allow replacing calls to error_free() with g_autoptr(Error) >>> declarations. >>> >>> Signed-off-by: Paolo Bonzini <[email protected]> >>> --- >>> include/qapi/error.h | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/include/qapi/error.h b/include/qapi/error.h >>> index 4a9260b0cc..8564657baf 100644 >>> --- a/include/qapi/error.h >>> +++ b/include/qapi/error.h >>> @@ -437,6 +437,8 @@ Error *error_copy(const Error *err); >>> */ >>> void error_free(Error *err); >>> +G_DEFINE_AUTOPTR_CLEANUP_FUNC(Error, error_free); >>> + >>> /* >>> * Convenience function to assert that *@errp is set, then silently free >>> it. >>> */ >> I'd like to see at least one actual use. > > I'll have one soon, I'll Cc you on that one. (I wrote this because > Dan suggested using g_autoptr(Error) in a review, but it doesn't work > yet).
I recommend to squash this patch into its first user, or maybe put it right before it.
