|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [PATCH] qom/object: Comment to use g_slist_free on object_class_get_list result |
Date: | Fri, 28 Feb 2020 11:08:58 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 2/28/20 11:06 AM, Marc-André Lureau wrote:
Hi On Fri, Feb 28, 2020 at 10:47 AM Daniel P. Berrangé <address@hidden> wrote:On Thu, Feb 27, 2020 at 08:09:42PM +0100, Philippe Mathieu-Daudé wrote:Document the list returned by object_class_get_list() must be released with g_slist_free() to avoid memory leaks. Signed-off-by: Philippe Mathieu-Daudé <address@hidden> --- include/qom/object.h | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/include/qom/object.h b/include/qom/object.h index 29546496c1..5517b56508 100644 --- a/include/qom/object.h +++ b/include/qom/object.h @@ -984,6 +984,9 @@ void object_class_foreach(void (*fn)(ObjectClass *klass, void *opaque), * @include_abstract: Whether to include abstract classes. * * Returns: A singly-linked list of the classes in reverse hashtable order. + * + * The returned list must be released with g_slist_free() + * when no longer required.I'd suggest "The returned list, but not its elements, must be released with g_slist_free() or g_autoptr when no longer required"As gobject-introspection annotations: "Returns: (transfer container) (element-type ObjectClass): A list of #ObjectClass"
Are you suggesting to replace "Returns: A singly-linked list of the classes in reverse hashtable order." by the line you quoted?
*/ GSList *object_class_get_list(const char *implements_type, bool include_abstract); @@ -995,6 +998,9 @@ GSList *object_class_get_list(const char *implements_type, * * Returns: A singly-linked list of the classes in alphabetical * case-insensitive order. + * + * The returned list must be released with g_slist_free() + * when no longer required. */ GSList *object_class_get_list_sorted(const char *implements_type, bool include_abstract); -- 2.21.1Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
[Prev in Thread] | Current Thread | [Next in Thread] |