qemu-trivial
[Top][All Lists]
Advanced

[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, 12 Nov 2019 10:33:37 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1

Le 05/11/2019 à 15:05, Laurent Vivier a écrit :
> 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
> 

Applied to my trivial-patches branch again.

Thanks,
Laurent




reply via email to

[Prev in Thread] Current Thread [Next in Thread]