[libvirt] [libvirt-glib 17/23] Remove GVirConfigObject::doc

Christophe Fergeau cfergeau at redhat.com
Fri Oct 7 09:41:02 UTC 2011


Now that the parsing is done before creating the object, this
member is no longer useful. Moreover, it's redundant with the ::node
member. Remove it and rely on always having an xmlNodePtr available
to represent the config object.
---
 libvirt-gconfig/libvirt-gconfig-object.c |   30 ------------------------------
 libvirt-gconfig/libvirt-gconfig-object.h |    1 -
 libvirt-gconfig/libvirt-gconfig.sym      |    1 -
 3 files changed, 0 insertions(+), 32 deletions(-)

diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c
index 97ac2e3..6fd5473 100644
--- a/libvirt-gconfig/libvirt-gconfig-object.c
+++ b/libvirt-gconfig/libvirt-gconfig-object.c
@@ -40,7 +40,6 @@ gboolean debugFlag;
 
 struct _GVirConfigObjectPrivate
 {
-    gchar *doc;
     gchar *schema;
 
     xmlNodePtr node;
@@ -50,7 +49,6 @@ G_DEFINE_ABSTRACT_TYPE(GVirConfigObject, gvir_config_object, G_TYPE_OBJECT);
 
 enum {
     PROP_0,
-    PROP_DOC,
     PROP_SCHEMA,
     PROP_NODE
 };
@@ -77,10 +75,6 @@ static void gvir_config_object_get_property(GObject *object,
     GVirConfigObjectPrivate *priv = conn->priv;
 
     switch (prop_id) {
-    case PROP_DOC:
-        g_value_set_string(value, priv->doc);
-        break;
-
     case PROP_SCHEMA:
         g_value_set_string(value, priv->schema);
         break;
@@ -103,11 +97,6 @@ static void gvir_config_object_set_property(GObject *object,
     GVirConfigObjectPrivate *priv = conn->priv;
 
     switch (prop_id) {
-    case PROP_DOC:
-        g_free(priv->doc);
-        priv->doc = g_value_dup_string(value);
-        break;
-
     case PROP_SCHEMA:
         g_free(priv->schema);
         priv->schema = g_value_dup_string(value);
@@ -138,7 +127,6 @@ static void gvir_config_object_finalize(GObject *object)
 
     DEBUG("Finalize GVirConfigObject=%p", conn);
 
-    g_free(priv->doc);
     g_free(priv->schema);
 
     /* FIXME: all objects describing a given XML document will share the
@@ -161,18 +149,6 @@ static void gvir_config_object_class_init(GVirConfigObjectClass *klass)
     object_class->set_property = gvir_config_object_set_property;
 
     g_object_class_install_property(object_class,
-                                    PROP_DOC,
-                                    g_param_spec_string("doc",
-                                                        "Doc",
-                                                        "The XML document",
-                                                        NULL,
-                                                        G_PARAM_READABLE |
-                                                        G_PARAM_WRITABLE |
-                                                        G_PARAM_CONSTRUCT_ONLY |
-                                                        G_PARAM_STATIC_NAME |
-                                                        G_PARAM_STATIC_NICK |
-                                                        G_PARAM_STATIC_BLURB));
-    g_object_class_install_property(object_class,
                                     PROP_SCHEMA,
                                     g_param_spec_string("schema",
                                                         "Schema",
@@ -291,12 +267,6 @@ gchar *gvir_config_object_to_xml(GVirConfigObject *config)
     return output_doc;
 }
 
-const gchar *gvir_config_object_get_doc(GVirConfigObject *config)
-{
-    GVirConfigObjectPrivate *priv = config->priv;
-    return priv->doc;
-}
-
 const gchar *gvir_config_object_get_schema(GVirConfigObject *config)
 {
     GVirConfigObjectPrivate *priv = config->priv;
diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h
index 5860102..133ca9f 100644
--- a/libvirt-gconfig/libvirt-gconfig-object.h
+++ b/libvirt-gconfig/libvirt-gconfig-object.h
@@ -64,7 +64,6 @@ void gvir_config_object_validate(GVirConfigObject *config,
 
 gchar *gvir_config_object_to_xml(GVirConfigObject *config);
 
-const gchar *gvir_config_object_get_doc(GVirConfigObject *config);
 const gchar *gvir_config_object_get_schema(GVirConfigObject *config);
 xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config);
 
diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym
index 94e596e..b67d152 100644
--- a/libvirt-gconfig/libvirt-gconfig.sym
+++ b/libvirt-gconfig/libvirt-gconfig.sym
@@ -27,7 +27,6 @@ LIBVIRT_GOBJECT_0.0.1 {
 	gvir_config_object_get_type;
 	gvir_config_object_error_quark;
 	gvir_config_object_new;
-	gvir_config_object_get_doc;
 	gvir_config_object_get_schema;
 	gvir_config_object_get_xml_node;
 	gvir_config_object_to_xml;
-- 
1.7.6.4




More information about the libvir-list mailing list