[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH qom v1 2/2] memory: remove object_property_add_c
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH qom v1 2/2] memory: remove object_property_add_child_array |
Date: |
Thu, 31 Jul 2014 11:38:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
Il 31/07/2014 07:35, Peter Crosthwaite ha scritto:
> Obsoleted by automatic object_property_add arrayification.
>
> Signed-off-by: Peter Crosthwaite <address@hidden>
> ---
>
> memory.c | 30 +++++-------------------------
> 1 file changed, 5 insertions(+), 25 deletions(-)
>
> diff --git a/memory.c b/memory.c
> index 64d7176..5272bf9 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -877,30 +877,6 @@ static char *memory_region_escape_name(const char *name)
> return escaped;
> }
>
> -static void object_property_add_child_array(Object *owner,
> - const char *name,
> - Object *child)
> -{
> - int i;
> - char *base_name = memory_region_escape_name(name);
> -
> - for (i = 0; ; i++) {
> - char *full_name = g_strdup_printf("%s[%d]", base_name, i);
> - Error *local_err = NULL;
> -
> - object_property_add_child(owner, full_name, child, &local_err);
> - g_free(full_name);
> - if (!local_err) {
> - break;
> - }
> -
> - error_free(local_err);
> - }
> -
> - g_free(base_name);
> -}
> -
> -
> void memory_region_init(MemoryRegion *mr,
> Object *owner,
> const char *name,
> @@ -918,8 +894,12 @@ void memory_region_init(MemoryRegion *mr,
> mr->name = g_strdup(name);
>
> if (name) {
> - object_property_add_child_array(owner, name, OBJECT(mr));
> + char *escaped_name = memory_region_escape_name(name);
> + char *name_array = g_strdup_printf("%s[*]", escaped_name);
> + object_property_add_child(owner, name_array, OBJECT(mr),
> &error_abort);
> object_unref(OBJECT(mr));
> + g_free(name_array);
> + g_free(escaped_name);
> }
> }
>
>
Reviewed-by: Paolo Bonzini <address@hidden>