[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qom: Fix error message in object_class_property_add()
From: |
Laurent Vivier |
Subject: |
Re: [PATCH] qom: Fix error message in object_class_property_add() |
Date: |
Tue, 5 Nov 2019 15:05:09 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 |
Le 04/11/2019 à 14:23, Greg Kurz a écrit :
> The error message in object_class_property_add() was copied from
> object_property_add() in commit 16bf7f522a2ff. Clarify that it is
> about a class, not an object.
>
> While here, have the format string in both functions to fit in a
> single line for better grep-ability, despite the checkpatch warning.
>
> Signed-off-by: Greg Kurz <address@hidden>
> ---
> qom/object.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/qom/object.c b/qom/object.c
> index 6fa9c619fac4..d51b57fba11e 100644
> --- a/qom/object.c
> +++ b/qom/object.c
> @@ -1106,9 +1106,8 @@ object_property_add(Object *obj, const char *name,
> const char *type,
> }
>
> if (object_property_find(obj, name, NULL) != NULL) {
> - error_setg(errp, "attempt to add duplicate property '%s'"
> - " to object (type '%s')", name,
> - object_get_typename(obj));
> + error_setg(errp, "attempt to add duplicate property '%s' to object
> (type '%s')",
> + name, object_get_typename(obj));
> return NULL;
> }
>
> @@ -1139,9 +1138,8 @@ object_class_property_add(ObjectClass *klass,
> ObjectProperty *prop;
>
> if (object_class_property_find(klass, name, NULL) != NULL) {
> - error_setg(errp, "attempt to add duplicate property '%s'"
> - " to object (type '%s')", name,
> - object_class_get_name(klass));
> + error_setg(errp, "attempt to add duplicate property '%s' to class
> (type '%s')",
> + name, object_class_get_name(klass));
> return NULL;
> }
>
>
>
Applied to my trivial-patches branch.
Thanks,
Laurent