From: Peter Xu Date: Mon, 1 Dec 2025 19:45:05 +0000 (-0500) Subject: Revert "error: define g_autoptr() cleanup function for the Error type" X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=aa0b2ec50be17d2f1ce3180f0b69a943f7c9c25b;p=thirdparty%2Fqemu.git Revert "error: define g_autoptr() cleanup function for the Error type" This reverts commit 18eb55546a54e443d94a4c49286348176ad4b00a. Due to the nature of how Error should be used (normally ownership will be passed over to Error APIs, like error_report_err), auto-free functions may be error prone on its own. The auto cleanup function was merged without proper review as pointed out by Dan and Markus: https://lore.kernel.org/r/aSWSLMi6ZhTCS_p2@redhat.com Cc: Cédric Le Goater Acked-by: Maciej S. Szmigiero Reviewed-by: Markus Armbruster Reviewed-by: Cédric Le Goater Acked-by: Maciej S. Szmigiero Link: https://lore.kernel.org/r/20251201194510.1121221-3-peterx@redhat.com Signed-off-by: Peter Xu --- diff --git a/include/qapi/error.h b/include/qapi/error.h index b16c6303f8..f3ce4a4a2d 100644 --- a/include/qapi/error.h +++ b/include/qapi/error.h @@ -437,8 +437,6 @@ 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. */