[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qmp: set accurate parameter name for error msg
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] qmp: set accurate parameter name for error msg of device-list-properties |
Date: |
Tue, 24 Jan 2017 18:34:42 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 01/24/2017 04:53 AM, Lin Ma wrote:
>> Signed-off-by: Lin Ma <address@hidden>
>> ---
>
> The commit message is rather sparse; is there an easy formula for
> triggering the error message, to give more context about the change?
Try this:
-> { "execute": "device-list-properties", "arguments": { "typename": "accel" } }
<- {"error": {"class": "GenericError", "desc": "Parameter 'name' expects
device"}}
Type "accel" exists, but it's not a subtype of TYPE_DEVICE.
Perhaps:
qmp: Fix argument name in error message of device-list-properties
The argument is called "typename", not "name".
>> qmp.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qmp.c b/qmp.c
>> index 0028f0b..812db6c 100644
>> --- a/qmp.c
>> +++ b/qmp.c
>> @@ -531,7 +531,7 @@ DevicePropertyInfoList *qmp_device_list_properties(const
>> char *typename,
>>
>> klass = object_class_dynamic_cast(klass, TYPE_DEVICE);
>> if (klass == NULL) {
>> - error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "name", TYPE_DEVICE);
>> + error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "typename",
>> TYPE_DEVICE);
>> return NULL;
>> }
Incomplete, the next one needs the same change:
if (object_class_is_abstract(klass)) {
error_setg(errp, QERR_INVALID_PARAMETER_VALUE, "name",
"non-abstract device type");
return NULL;
}
Reproducer:
-> { "execute": "device-list-properties", "arguments": { "typename": "pci-qxl"
} }
<- {"error": {"class": "GenericError", "desc": "Parameter 'name' expects
non-abstract device type"}}
Please fix this one, too.