[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/10] qdev: Use error_prepend() for errors appl
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 04/10] qdev: Use error_prepend() for errors applying globals |
Date: |
Mon, 20 Jun 2016 10:02:38 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eduardo Habkost <address@hidden> writes:
> The same Error* will be used in an error_propagate() call in the
> future, so prepend a "can't apply global" prefix to it.
What future? A future patch?
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> hw/core/qdev-properties.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/core/qdev-properties.c b/hw/core/qdev-properties.c
> index 64e17aa..cd19603 100644
> --- a/hw/core/qdev-properties.c
> +++ b/hw/core/qdev-properties.c
> @@ -1079,8 +1079,9 @@ static void qdev_prop_set_globals_for_type(DeviceState
> *dev,
> object_property_parse(OBJECT(dev), prop->value, prop->property,
> &err);
> if (err != NULL) {
> assert(prop->user_provided);
> - error_reportf_err(err, "Warning: global %s.%s=%s ignored: ",
> - prop->driver, prop->property, prop->value);
> + error_prepend(&err, "can't apply global %s.%s=%s: ",
> + prop->driver, prop->property, prop->value);
> + error_reportf_err(err, "Warning: ");
> }
> }
> }
You reword the error message. Should be mentioned in the commit
message.
Why do you need to prepend the "Warning: ..." in two steps, first
error_prepend() for the "...", then error_reportf_err() for "Warning: "?
Perhaps it'll become clear later in the series, but it's not obvious
now.
- Re: [Qemu-devel] [PATCH 01/10] qdev: Don't stop applying globals on first error, (continued)
- [Qemu-devel] [PATCH 05/10] qdev: GlobalProperty.errp field, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 02/10] qdev: Eliminate qemu_add_globals() function, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 07/10] vl: Set errp to &error_abort on machine compat_props, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 08/10] qdev: Eliminate "global not used" warning, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 04/10] qdev: Use error_prepend() for errors applying globals, Eduardo Habkost, 2016/06/15
- Re: [Qemu-devel] [PATCH 04/10] qdev: Use error_prepend() for errors applying globals,
Markus Armbruster <=
- [Qemu-devel] [PATCH 06/10] machine: Add machine_register_compat_props() function, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 10/10] machine: Skip global registration for non-existing classes, Eduardo Habkost, 2016/06/15
- [Qemu-devel] [PATCH 09/10] qdev: Eliminate GlobalProperty 'used' and 'user_provided' fields, Eduardo Habkost, 2016/06/15
- Re: [Qemu-devel] [PATCH 00/10] globals: Clean up validation and error checking, Igor Mammedov, 2016/06/20