[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 07/16] qdev: allow reusing get/set for legacy pr
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 07/16] qdev: allow reusing get/set for legacy property |
Date: |
Thu, 02 Feb 2012 23:38:49 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111220 Thunderbird/9.0 |
Am 02.02.2012 17:45, schrieb Paolo Bonzini:
> In some cases, a legacy property does need a special print method
> but not a special parse method. In this case, we can reuse the get/set
> from the static (non-legacy) property.
>
> If neither parse nor print is needed, though, do not register the
> legacy property at all. The previous patch ensures that the right
> fallback will be used.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/qdev-monitor.c | 5 ++---
> hw/qdev-properties.c | 6 +++---
> hw/qdev.c | 11 +++++++----
> 3 files changed, 12 insertions(+), 10 deletions(-)
>
> diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor.c
> index 64505b4..e21bd50 100644
> --- a/hw/qdev-monitor.c
> +++ b/hw/qdev-monitor.c
> @@ -489,8 +489,8 @@ static void qdev_print_props(Monitor *mon, DeviceState
> *dev, Property *props,
> {
> if (!props)
> return;
> - while (props->name) {
> - Error *err;
> + for (; props->name; props++) {
> + Error *err = NULL;
Do either of these fix a bug? Should be mentioned or avoided.
> char *value;
> char *legacy_name = g_strdup_printf("legacy-%s", props->name);
> if (object_property_get_type(OBJECT(dev), legacy_name, NULL)) {
> @@ -507,7 +507,6 @@ static void qdev_print_props(Monitor *mon, DeviceState
> *dev, Property *props,
> qdev_printf("%s-prop: %s = %s\n", prefix, props->name,
> value && *value ? value : "<null>");
> g_free(value);
> - props++;
> }
> }
>
> diff --git a/hw/qdev-properties.c b/hw/qdev-properties.c
> index 7c41140..16f9b22 100644
> --- a/hw/qdev-properties.c
> +++ b/hw/qdev-properties.c
> @@ -1025,13 +1025,13 @@ void error_set_from_qdev_prop_error(Error **errp, int
> ret, DeviceState *dev,
> int qdev_prop_parse(DeviceState *dev, const char *name, const char *value)
> {
> char *legacy_name;
> - Error *err;
> + Error *err = NULL;
>
> legacy_name = g_strdup_printf("legacy-%s", name);
> if (object_property_get_type(OBJECT(dev), legacy_name, NULL)) {
> - object_property_set_str(OBJECT(dev), legacy_name, value, &err);
> + object_property_set_str(OBJECT(dev), value, legacy_name, &err);
> } else {
> - object_property_set_str(OBJECT(dev), name, value, &err);
> + object_property_set_str(OBJECT(dev), value, name, &err);
> }
Bugfix?
> g_free(legacy_name);
>
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Qemu-devel] [PATCH 03/16] qom: do not include qdev header file, Paolo Bonzini, 2012/02/02
[Qemu-devel] [PATCH 05/16] qom: add property get/set wrappers for C types, Paolo Bonzini, 2012/02/02
[Qemu-devel] [PATCH 06/16] qdev: remove direct calls to print/parse, Paolo Bonzini, 2012/02/02
[Qemu-devel] [PATCH 09/16] qdev: remove parse/print methods for mac properties, Paolo Bonzini, 2012/02/02
[Qemu-devel] [PATCH 07/16] qdev: allow reusing get/set for legacy property, Paolo Bonzini, 2012/02/02
- Re: [Qemu-devel] [PATCH 07/16] qdev: allow reusing get/set for legacy property,
Andreas Färber <=
[Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/02
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Anthony Liguori, 2012/02/02
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/02
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Anthony Liguori, 2012/02/03
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/03
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/03
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Anthony Liguori, 2012/02/03
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/04
- Re: [Qemu-devel] [PATCH 10/16] qdev: make the non-legacy pci address property accept an integer, Paolo Bonzini, 2012/02/04