qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: qemu-devel@nongnu.org
Cc: pbonzini@redhat.com, peter.crosthwaite@xilinx.com, afaerber@suse.de
Subject: [Qemu-devel] [PATCH 1/4] qom: New object_gen_new_property_name()
Date: Wed, 12 Nov 2014 18:08:47 +0100	[thread overview]
Message-ID: <1415812130-2592-2-git-send-email-armbru@redhat.com> (raw)
In-Reply-To: <1415812130-2592-1-git-send-email-armbru@redhat.com>

The next few commits will use it to replace object_property_add()'s
"automatic arrayification" (commit 3396590).  "Automatic
arrayification" is a convenience feature for creating a bunch of
properties with a common type, accessors and so forth, named in a
peculiar way: "foo[0]", "foo[1]", ...  It's implemented by making
property names ending with "[*]" magical.  The magic is uncalled for,
as names can be just as well generated separately from adding
properties.

The name object_gen_new_property_name() is exceedingly long, but
that's how QOM names are.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 include/qom/object.h |  8 ++++++++
 qom/object.c         | 14 ++++++++++++++
 2 files changed, 22 insertions(+)

diff --git a/include/qom/object.h b/include/qom/object.h
index 89c3092..b5c9201 100644
--- a/include/qom/object.h
+++ b/include/qom/object.h
@@ -832,6 +832,14 @@ void object_property_del(Object *obj, const char *name, Error **errp);
 ObjectProperty *object_property_find(Object *obj, const char *name,
                                      Error **errp);
 
+/*
+ * Return a property name that doesn't clash with @obj's existing ones.
+ * The name is of the form %s[%d], where %s is @stem, and %d counts up
+ * from zero.
+ * The caller should free the name.
+ */
+char *object_gen_new_property_name(Object *obj, const char *stem);
+
 void object_unparent(Object *obj);
 
 /**
diff --git a/qom/object.c b/qom/object.c
index 1812c73..4c46662 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -808,6 +808,20 @@ void object_property_del(Object *obj, const char *name, Error **errp)
     g_free(prop);
 }
 
+char *object_gen_new_property_name(Object *obj, const char *stem)
+{
+    int i;
+
+    for (i = 0; ; ++i) {
+        char *name = g_strdup_printf("%s[%d]", stem, i);
+
+        if (!object_property_find(obj, name, NULL)) {
+            return name;
+        }
+        g_free(name);
+    }
+}
+
 void object_property_get(Object *obj, Visitor *v, const char *name,
                          Error **errp)
 {
-- 
1.9.3

  reply	other threads:[~2014-11-12 17:09 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-12 17:08 [Qemu-devel] [PATCH 0/4] qom: Replace "automatic arrayification" Markus Armbruster
2014-11-12 17:08 ` Markus Armbruster [this message]
2014-11-12 17:08 ` [Qemu-devel] [PATCH 2/4] memory: Use object_gen_new_property_name() instead of "arrayification" Markus Armbruster
2014-11-12 17:08 ` [Qemu-devel] [PATCH 3/4] qdev: " Markus Armbruster
2014-11-12 17:08 ` [Qemu-devel] [PATCH 4/4] Revert "qom: Add automatic arrayification to object_property_add()" Markus Armbruster
2014-11-12 17:14   ` Andreas Färber
2014-11-12 22:25     ` Paolo Bonzini
2014-11-12 22:47       ` Peter Maydell
2014-11-13 10:05         ` Paolo Bonzini
2014-11-13 10:50           ` Peter Maydell
2014-11-12 17:13 ` [Qemu-devel] [PATCH 0/4] qom: Replace "automatic arrayification" Paolo Bonzini
2014-11-13  8:08   ` Markus Armbruster
2014-11-13 10:08     ` Paolo Bonzini
2014-11-13 14:34       ` Markus Armbruster
2014-11-13 14:39         ` Paolo Bonzini
2014-11-14 14:21           ` Markus Armbruster
2014-11-14 14:25             ` Paolo Bonzini
2014-11-14 15:10               ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1415812130-2592-2-git-send-email-armbru@redhat.com \
    --to=armbru@redhat.com \
    --cc=afaerber@suse.de \
    --cc=pbonzini@redhat.com \
    --cc=peter.crosthwaite@xilinx.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).