These helper functions will be useful later on. --- examples/Makefile.am | 1 + libvirt-gconfig/Makefile.am | 3 + libvirt-gconfig/libvirt-gconfig-helpers.c | 179 +++++++++++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-helpers.h | 48 ++++++++ libvirt-gconfig/libvirt-gconfig-object.c | 52 --------- libvirt-gconfig/libvirt-gconfig.h | 2 + libvirt-gconfig/libvirt-gconfig.sym | 1 + libvirt-gobject/Makefile.am | 2 + 8 files changed, 236 insertions(+), 52 deletions(-) create mode 100644 libvirt-gconfig/libvirt-gconfig-helpers.c create mode 100644 libvirt-gconfig/libvirt-gconfig-helpers.h diff --git a/libvirt-gconfig/libvirt-gconfig-helpers.h b/libvirt-gconfig/libvirt-gconfig-helpers.h new file mode 100644 index 0000000..1bdf2d0 --- /dev/null +++ b/libvirt-gconfig/libvirt-gconfig-helpers.h @@ -0,0 +1,48 @@

+ +GQuark gvir_config_object_error_quark(void); +#define GVIR_CONFIG_OBJECT_ERROR gvir_config_object_error_quark() + +GError *gvir_xml_error_new(GQuark domain, gint code, + const gchar *format, ...); +xmlNodePtr gvir_config_xml_parse(const char *xml, + const char *root_node, + GError **err); +xmlNode * gvir_config_xml_get_element (xmlNode *node, ...); +xmlChar * gvir_config_xml_get_child_element_content (xmlNode *node, + const char *child_name); +char *gvir_config_xml_get_child_element_content_glib (xmlNode *node, + const char *child_name);

diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym index 2d09c37..24407c3 100644 --- a/libvirt-gconfig/libvirt-gconfig.sym +++ b/libvirt-gconfig/libvirt-gconfig.sym @@ -22,6 +22,7 @@ LIBVIRT_GOBJECT_0.0.1 { gvir_config_node_device_new; gvir_config_object_get_type; + gvir_config_object_error_quark;

Hi, On Tue, Oct 18, 2011 at 12:06:50PM +0100, Daniel P. Berrange wrote: > On Fri, Oct 07, 2011 at 11:40:46AM +0200, Christophe Fergeau wrote: > > These helper functions will be useful later on. > > --- > > examples/Makefile.am | 1 + > > libvirt-gconfig/Makefile.am | 3 + > > libvirt-gconfig/libvirt-gconfig-helpers.c | 179 +++++++++++++++++++++++++++++ > > libvirt-gconfig/libvirt-gconfig-helpers.h | 48 ++++++++ > > libvirt-gconfig/libvirt-gconfig-object.c | 52 --------- > > libvirt-gconfig/libvirt-gconfig.h | 2 + > > libvirt-gconfig/libvirt-gconfig.sym | 1 + > > libvirt-gobject/Makefile.am | 2 + > > 8 files changed, 236 insertions(+), 52 deletions(-) > > create mode 100644 libvirt-gconfig/libvirt-gconfig-helpers.c > > create mode 100644 libvirt-gconfig/libvirt-gconfig-helpers.h > > > > diff --git a/libvirt-gconfig/libvirt-gconfig-helpers.h b/libvirt-gconfig/libvirt-gconfig-helpers.h > > new file mode 100644 > > index 0000000..1bdf2d0 > > --- /dev/null > > +++ b/libvirt-gconfig/libvirt-gconfig-helpers.h > > @@ -0,0 +1,48 @@ > > > + > > +GQuark gvir_config_object_error_quark(void); > > +#define GVIR_CONFIG_OBJECT_ERROR gvir_config_object_error_quark() > > + > > +GError *gvir_xml_error_new(GQuark domain, gint code, > > + const gchar *format, ...); > > +xmlNodePtr gvir_config_xml_parse(const char *xml, > > + const char *root_node, > > + GError **err); > > +xmlNode * gvir_config_xml_get_element (xmlNode *node, ...); > > +xmlChar * gvir_config_xml_get_child_element_content (xmlNode *node, > > + const char *child_name); > > +char *gvir_config_xml_get_child_element_content_glib (xmlNode *node, > > + const char *child_name); > > > > diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym > > index 2d09c37..24407c3 100644 > > --- a/libvirt-gconfig/libvirt-gconfig.sym > > +++ b/libvirt-gconfig/libvirt-gconfig.sym > > @@ -22,6 +22,7 @@ LIBVIRT_GOBJECT_0.0.1 { > > gvir_config_node_device_new; > > > > gvir_config_object_get_type; > > + gvir_config_object_error_quark; > > > Don't we want to also export the rest of those new APIs in > gconfig-helpers.h ? > > > ACK, if you tell me why they're not required, or if you add them > before pushing the patch. At this point, I'm not sure what kind of low-level API we want to make available to the library user (by low-level, I mean letting people directly poke at the xmlNode wrapped in a GVirConfigObject instance). That's why I chose not to export them for now, and to make them public if/once they are needed. If you think they should be exported, I'll make the change.

Report an error when trying to parse the current XML document associated with the GVirConfig object when no document has been set. --- libvirt-gconfig/libvirt-gconfig-object.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index 4a58323..6f80346 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -187,6 +187,14 @@ gvir_config_object_parse(GVirConfigObject *config, if (priv->docHandle) return; + if (!priv->doc) { + *err = g_error_new(GVIR_CONFIG_OBJECT_ERROR, + 0, + "%s", + "No XML document to parse"); + return; + } + priv->docHandle = xmlParseMemory(priv->doc, strlen(priv->doc)); if (!priv->docHandle) { *err = gvir_xml_error_new(GVIR_CONFIG_OBJECT_ERROR,

On Fri, Oct 07, 2011 at 11:40:49AM +0200, Christophe Fergeau wrote: > @@ -277,6 +306,8 @@ const gchar *gvir_config_object_get_schema(GVirConfigObject *config) > } > > /* NB: the xmlDocPtr must not be freed by the caller */ > +/* gupnp has wrapped xmlDoc in a gobject */ > +/* not really useful, can be obtained from xmlNode::doc */ I guess the reason for doing that would be to avoid any libxml2 data types in the public API. Might be worth considering in the future.

On Fri, Oct 07, 2011 at 11:40:53AM +0200, Christophe Fergeau wrote: > --- > libvirt-gconfig/libvirt-gconfig-domain.c | 27 +++++++++++++++++++++++++++ > libvirt-gconfig/libvirt-gconfig-domain.h | 1 + > libvirt-gconfig/libvirt-gconfig.sym | 1 + > 3 files changed, 29 insertions(+), 0 deletions(-) > > diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c > index 7ae10b8..06fc3fa 100644 > --- a/libvirt-gconfig/libvirt-gconfig-domain.c > +++ b/libvirt-gconfig/libvirt-gconfig-domain.c > @@ -130,3 +130,30 @@ char *gvir_config_domain_get_name(GVirConfigDomain *domain) > return gvir_config_xml_get_child_element_content_glib(node, "name"); > > } > + > +void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) > +{ > + xmlNodePtr parent_node; > + xmlNodePtr old_node; > + xmlNodePtr new_node; > + xmlChar *encoded_name; > + > + parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain), > + NULL); > + encoded_name = xmlEncodeEntitiesReentrant(parent_node->doc, > + (xmlChar *)name); > + new_node = xmlNewDocNode(parent_node->doc, NULL, > + (xmlChar *)"name", encoded_name); > + xmlFree(encoded_name); > + > + old_node = gvir_config_xml_get_element(parent_node, "name", NULL); > + > + if (old_node) { > + old_node = xmlReplaceNode(old_node, new_node); > + xmlFreeNode(old_node); > + } else { > + xmlAddChild(parent_node, new_node); > + } > + > + g_object_notify(G_OBJECT(domain), "name"); > +} Perhaps we want another helper method for setting XML elements too.

In python-virtinst, they had a helper which allowed a very simple xpath like use.eg gvir_config_xml_set_element(parent_node, "/name", node); Or gvir_config_xml_set_element(parent_node, "/foo/bar/name", node); automagically creating foo & bar in the process.

And setting NULL terminated list of attributes gvir_config_xml_set_element_attrs(parent_node, "/blah", "someattr", "somevalue", "otherattr", "othervalue", NULL);

--- libvirt-gconfig/libvirt-gconfig-domain.c | 19 ++++++++++++++++++- 1 files changed, 18 insertions(+), 1 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 06fc3fa..00cab80 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -63,6 +63,22 @@ static void gvir_config_domain_get_property(GObject *object, } } +static void gvir_config_domain_set_property(GObject *object, + guint prop_id, + const GValue *value, + GParamSpec *pspec) +{ + GVirConfigDomain *domain = GVIR_CONFIG_DOMAIN(object); + + switch (prop_id) { + case PROP_NAME: + gvir_config_domain_set_name(domain, g_value_get_string(value)); + break; + default: + G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec); + } +} + static void gvir_config_domain_class_init(GVirConfigDomainClass *klass) { @@ -71,6 +87,7 @@ static void gvir_config_domain_class_init(GVirConfigDomainClass *klass) g_type_class_add_private(klass, sizeof(GVirConfigDomainPrivate)); object_class->get_property = gvir_config_domain_get_property; + object_class->set_property = gvir_config_domain_set_property; g_object_class_install_property(object_class, PROP_NAME, @@ -78,7 +95,7 @@ static void gvir_config_domain_class_init(GVirConfigDomainClass *klass) "Name", "Domain Name", NULL, - G_PARAM_READABLE | + G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); }

--- libvirt-gconfig/libvirt-gconfig-object.c | 19 +++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-object.h | 2 ++ libvirt-gconfig/libvirt-gconfig.sym | 1 + 3 files changed, 22 insertions(+), 0 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index 762baf9..66102ec 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -293,6 +293,25 @@ void gvir_config_object_validate(GVirConfigObject *config, xmlRelaxNGFree(rng); } +gchar *gvir_config_object_to_xml(GVirConfigObject *config) +{ + xmlChar *doc; + int size; + xmlNodePtr node; + gchar *output_doc; + + node = gvir_config_object_get_xml_node(config, NULL); + if (node == NULL) + return NULL; + + xmlDocDumpMemory(node->doc, &doc, &size); + + output_doc = g_strdup((gchar *)doc); + xmlFree(doc); + + return output_doc; +} + const gchar *gvir_config_object_get_doc(GVirConfigObject *config) { GVirConfigObjectPrivate *priv = config->priv; diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index a0c0489..f6ac28b 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -62,6 +62,8 @@ GType gvir_config_object_get_type(void); void gvir_config_object_validate(GVirConfigObject *config, GError **err); +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); xmlDocPtr gvir_config_object_get_xml_doc(GVirConfigObject *config, GError **error); diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym index 2571c05..5c4492e 100644 --- a/libvirt-gconfig/libvirt-gconfig.sym +++ b/libvirt-gconfig/libvirt-gconfig.sym @@ -31,6 +31,7 @@ LIBVIRT_GOBJECT_0.0.1 { gvir_config_object_get_schema; gvir_config_object_get_xml_doc; gvir_config_object_get_xml_node; + gvir_config_object_to_xml; gvir_config_object_validate; gvir_config_secret_get_type;

s/gvir_config_object_get_doc/gvir_config_object_to_xml/ Prefer gvir_config_object_to_xml over gvir_config_object_get_doc since the latter will be removed soon. --- libvirt-gobject/libvirt-gobject-connection.c | 2 +- libvirt-gobject/libvirt-gobject-storage-pool.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libvirt-gobject/libvirt-gobject-connection.c b/libvirt-gobject/libvirt-gobject-connection.c index 007d86e..71fa768 100644 --- a/libvirt-gobject/libvirt-gobject-connection.c +++ b/libvirt-gobject/libvirt-gobject-connection.c @@ -1170,7 +1170,7 @@ GVirDomain *gvir_connection_create_domain(GVirConnection *conn, virDomainPtr handle; GVirConnectionPrivate *priv = conn->priv; - xml = gvir_config_object_get_doc(GVIR_CONFIG_OBJECT(conf)); + xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(conf)); g_return_val_if_fail(xml != NULL, NULL); diff --git a/libvirt-gobject/libvirt-gobject-storage-pool.c b/libvirt-gobject/libvirt-gobject-storage-pool.c index fc9cba9..8cc481e 100644 --- a/libvirt-gobject/libvirt-gobject-storage-pool.c +++ b/libvirt-gobject/libvirt-gobject-storage-pool.c @@ -491,7 +491,7 @@ GVirStorageVol *gvir_storage_pool_create_volume virStorageVolPtr handle; GVirStoragePoolPrivate *priv = pool->priv; - xml = gvir_config_object_get_doc(GVIR_CONFIG_OBJECT(conf)); + xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(conf)); g_return_val_if_fail(xml != NULL, NULL);

These functions now create empty config objects to be consistent with what gvir_config_domain_new does. --- libvirt-gconfig/libvirt-gconfig-capabilities.c | 10 ++++++++-- libvirt-gconfig/libvirt-gconfig-capabilities.h | 2 +- libvirt-gconfig/libvirt-gconfig-domain-snapshot.c | 9 +++++++-- libvirt-gconfig/libvirt-gconfig-domain-snapshot.h | 2 +- libvirt-gconfig/libvirt-gconfig-interface.c | 12 ++++++++---- libvirt-gconfig/libvirt-gconfig-interface.h | 2 +- libvirt-gconfig/libvirt-gconfig-network-filter.c | 8 ++++++-- libvirt-gconfig/libvirt-gconfig-network-filter.h | 2 +- libvirt-gconfig/libvirt-gconfig-network.c | 8 ++++++-- libvirt-gconfig/libvirt-gconfig-network.h | 2 +- libvirt-gconfig/libvirt-gconfig-node-device.c | 9 +++++++-- libvirt-gconfig/libvirt-gconfig-node-device.h | 2 +- libvirt-gconfig/libvirt-gconfig-secret.c | 8 ++++++-- libvirt-gconfig/libvirt-gconfig-secret.h | 2 +- libvirt-gconfig/libvirt-gconfig-storage-pool.c | 8 ++++++-- libvirt-gconfig/libvirt-gconfig-storage-pool.h | 2 +- libvirt-gconfig/libvirt-gconfig-storage-vol.c | 12 ++++++++---- libvirt-gconfig/libvirt-gconfig-storage-vol.h | 2 +- libvirt-gobject/libvirt-gobject-domain-snapshot.c | 3 +++ libvirt-gobject/libvirt-gobject-interface.c | 3 +++ libvirt-gobject/libvirt-gobject-network-filter.c | 3 +++ libvirt-gobject/libvirt-gobject-network.c | 3 +++ libvirt-gobject/libvirt-gobject-node-device.c | 4 ++++ libvirt-gobject/libvirt-gobject-secret.c | 4 ++++ libvirt-gobject/libvirt-gobject-storage-pool.c | 3 +++ libvirt-gobject/libvirt-gobject-storage-vol.c | 3 +++ 26 files changed, 97 insertions(+), 31 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-capabilities.c b/libvirt-gconfig/libvirt-gconfig-capabilities.c index b881a17..8ed0e2e 100644 --- a/libvirt-gconfig/libvirt-gconfig-capabilities.c +++ b/libvirt-gconfig/libvirt-gconfig-capabilities.c @@ -61,10 +61,16 @@ static void gvir_config_capabilities_init(GVirConfigCapabilities *conn) } -GVirConfigCapabilities *gvir_config_capabilities_new(const gchar *xml) +GVirConfigCapabilities *gvir_config_capabilities_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + /* FIXME: what is the XML root of the capability node? I suspect it is + * either 'guest' or 'host' */ + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"capabilities", NULL); return GVIR_CONFIG_CAPABILITIES(g_object_new(GVIR_TYPE_CONFIG_CAPABILITIES, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/capability.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-capabilities.h b/libvirt-gconfig/libvirt-gconfig-capabilities.h index 5b79cab..b5660df 100644 --- a/libvirt-gconfig/libvirt-gconfig-capabilities.h +++ b/libvirt-gconfig/libvirt-gconfig-capabilities.h @@ -59,7 +59,7 @@ struct _GVirConfigCapabilitiesClass GType gvir_config_capabilities_get_type(void); -GVirConfigCapabilities *gvir_config_capabilities_new(const gchar *xml); +GVirConfigCapabilities *gvir_config_capabilities_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-domain-snapshot.c b/libvirt-gconfig/libvirt-gconfig-domain-snapshot.c index 841f7af..35f63ec 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-snapshot.c +++ b/libvirt-gconfig/libvirt-gconfig-domain-snapshot.c @@ -61,10 +61,15 @@ static void gvir_config_domain_snapshot_init(GVirConfigDomainSnapshot *conn) } -GVirConfigDomainSnapshot *gvir_config_domain_snapshot_new(const gchar *xml) +GVirConfigDomainSnapshot *gvir_config_domain_snapshot_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"domainsnapshot", NULL); + return GVIR_CONFIG_DOMAIN_SNAPSHOT(g_object_new(GVIR_TYPE_CONFIG_DOMAIN_SNAPSHOT, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/domainsnapshot.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-domain-snapshot.h b/libvirt-gconfig/libvirt-gconfig-domain-snapshot.h index c6598c2..1924903 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain-snapshot.h +++ b/libvirt-gconfig/libvirt-gconfig-domain-snapshot.h @@ -59,7 +59,7 @@ struct _GVirConfigDomainSnapshotClass GType gvir_config_domain_snapshot_get_type(void); -GVirConfigDomainSnapshot *gvir_config_domain_snapshot_new(const gchar *xml); +GVirConfigDomainSnapshot *gvir_config_domain_snapshot_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-interface.c b/libvirt-gconfig/libvirt-gconfig-interface.c index 181ae52..48cbcfb 100644 --- a/libvirt-gconfig/libvirt-gconfig-interface.c +++ b/libvirt-gconfig/libvirt-gconfig-interface.c @@ -61,10 +61,14 @@ static void gvir_config_interface_init(GVirConfigInterface *conn) } -GVirConfigInterface *gvir_config_interface_new(const gchar *xml) +GVirConfigInterface *gvir_config_interface_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"interface", NULL); return GVIR_CONFIG_INTERFACE(g_object_new(GVIR_TYPE_CONFIG_INTERFACE, - "doc", xml, - "schema", DATADIR "/libvirt/schemas/interface.rng", - NULL)); + "node", doc->children, + "schema", DATADIR "/libvirt/schemas/interface.rng", + NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-interface.h b/libvirt-gconfig/libvirt-gconfig-interface.h index ca5e6cf..abfcc4e 100644 --- a/libvirt-gconfig/libvirt-gconfig-interface.h +++ b/libvirt-gconfig/libvirt-gconfig-interface.h @@ -59,7 +59,7 @@ struct _GVirConfigInterfaceClass GType gvir_config_interface_get_type(void); -GVirConfigInterface *gvir_config_interface_new(const gchar *xml); +GVirConfigInterface *gvir_config_interface_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-network-filter.c b/libvirt-gconfig/libvirt-gconfig-network-filter.c index 783a5ed..1b4c22d 100644 --- a/libvirt-gconfig/libvirt-gconfig-network-filter.c +++ b/libvirt-gconfig/libvirt-gconfig-network-filter.c @@ -61,10 +61,14 @@ static void gvir_config_network_filter_init(GVirConfigNetworkFilter *conn) } -GVirConfigNetworkFilter *gvir_config_network_filter_new(const gchar *xml) +GVirConfigNetworkFilter *gvir_config_network_filter_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"filter", NULL); return GVIR_CONFIG_NETWORK_FILTER(g_object_new(GVIR_TYPE_CONFIG_NETWORK_FILTER, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/nwfilter.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-network-filter.h b/libvirt-gconfig/libvirt-gconfig-network-filter.h index 0d50b2c..c3f87b7 100644 --- a/libvirt-gconfig/libvirt-gconfig-network-filter.h +++ b/libvirt-gconfig/libvirt-gconfig-network-filter.h @@ -59,7 +59,7 @@ struct _GVirConfigNetworkFilterClass GType gvir_config_network_filter_get_type(void); -GVirConfigNetworkFilter *gvir_config_network_filter_new(const gchar *xml); +GVirConfigNetworkFilter *gvir_config_network_filter_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-network.c b/libvirt-gconfig/libvirt-gconfig-network.c index a6310e3..f2d19f7 100644 --- a/libvirt-gconfig/libvirt-gconfig-network.c +++ b/libvirt-gconfig/libvirt-gconfig-network.c @@ -61,10 +61,14 @@ static void gvir_config_network_init(GVirConfigNetwork *conn) } -GVirConfigNetwork *gvir_config_network_new(const gchar *xml) +GVirConfigNetwork *gvir_config_network_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"network", NULL); return GVIR_CONFIG_NETWORK(g_object_new(GVIR_TYPE_CONFIG_NETWORK, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/network.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-network.h b/libvirt-gconfig/libvirt-gconfig-network.h index 79b80d0..f156971 100644 --- a/libvirt-gconfig/libvirt-gconfig-network.h +++ b/libvirt-gconfig/libvirt-gconfig-network.h @@ -59,7 +59,7 @@ struct _GVirConfigNetworkClass GType gvir_config_network_get_type(void); -GVirConfigNetwork *gvir_config_network_new(const gchar *xml); +GVirConfigNetwork *gvir_config_network_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-node-device.c b/libvirt-gconfig/libvirt-gconfig-node-device.c index 8f028ee..a6e6d9e 100644 --- a/libvirt-gconfig/libvirt-gconfig-node-device.c +++ b/libvirt-gconfig/libvirt-gconfig-node-device.c @@ -61,10 +61,15 @@ static void gvir_config_node_device_init(GVirConfigNodeDevice *conn) } -GVirConfigNodeDevice *gvir_config_node_device_new(const gchar *xml) +GVirConfigNodeDevice *gvir_config_node_device_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + /* FIXME: correct node name ? */ + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"device", NULL); return GVIR_CONFIG_NODE_DEVICE(g_object_new(GVIR_TYPE_CONFIG_NODE_DEVICE, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/nodedev.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-node-device.h b/libvirt-gconfig/libvirt-gconfig-node-device.h index d9ddacf..720b580 100644 --- a/libvirt-gconfig/libvirt-gconfig-node-device.h +++ b/libvirt-gconfig/libvirt-gconfig-node-device.h @@ -59,7 +59,7 @@ struct _GVirConfigNodeDeviceClass GType gvir_config_node_device_get_type(void); -GVirConfigNodeDevice *gvir_config_node_device_new(const gchar *xml); +GVirConfigNodeDevice *gvir_config_node_device_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-secret.c b/libvirt-gconfig/libvirt-gconfig-secret.c index 5b639a9..505de4c 100644 --- a/libvirt-gconfig/libvirt-gconfig-secret.c +++ b/libvirt-gconfig/libvirt-gconfig-secret.c @@ -61,10 +61,14 @@ static void gvir_config_secret_init(GVirConfigSecret *conn) } -GVirConfigSecret *gvir_config_secret_new(const gchar *xml) +GVirConfigSecret *gvir_config_secret_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"secret", NULL); return GVIR_CONFIG_SECRET(g_object_new(GVIR_TYPE_CONFIG_SECRET, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/secret.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-secret.h b/libvirt-gconfig/libvirt-gconfig-secret.h index 8163ef9..fbd4564 100644 --- a/libvirt-gconfig/libvirt-gconfig-secret.h +++ b/libvirt-gconfig/libvirt-gconfig-secret.h @@ -59,7 +59,7 @@ struct _GVirConfigSecretClass GType gvir_config_secret_get_type(void); -GVirConfigSecret *gvir_config_secret_new(const gchar *xml); +GVirConfigSecret *gvir_config_secret_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool.c b/libvirt-gconfig/libvirt-gconfig-storage-pool.c index 3d6ce5e..ef32aeb 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool.c @@ -61,10 +61,14 @@ static void gvir_config_storage_pool_init(GVirConfigStoragePool *conn) } -GVirConfigStoragePool *gvir_config_storage_pool_new(const gchar *xml) +GVirConfigStoragePool *gvir_config_storage_pool_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"pool", NULL); return GVIR_CONFIG_STORAGE_POOL(g_object_new(GVIR_TYPE_CONFIG_STORAGE_POOL, - "doc", xml, + "node", doc->children, "schema", DATADIR "/libvirt/schemas/storagepool.rng", NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-storage-pool.h b/libvirt-gconfig/libvirt-gconfig-storage-pool.h index fcf2fda..e5912f9 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-pool.h +++ b/libvirt-gconfig/libvirt-gconfig-storage-pool.h @@ -59,7 +59,7 @@ struct _GVirConfigStoragePoolClass GType gvir_config_storage_pool_get_type(void); -GVirConfigStoragePool *gvir_config_storage_pool_new(const gchar *xml); +GVirConfigStoragePool *gvir_config_storage_pool_new(void); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol.c b/libvirt-gconfig/libvirt-gconfig-storage-vol.c index d3ac8de..886b1e0 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol.c +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol.c @@ -61,10 +61,14 @@ static void gvir_config_storage_vol_init(GVirConfigStorageVol *conn) } -GVirConfigStorageVol *gvir_config_storage_vol_new(const gchar *xml) +GVirConfigStorageVol *gvir_config_storage_vol_new(void) { + xmlDocPtr doc; + + doc = xmlNewDoc((xmlChar *)"1.0"); + doc->children = xmlNewDocNode(doc, NULL, (xmlChar *)"volume", NULL); return GVIR_CONFIG_STORAGE_VOL(g_object_new(GVIR_TYPE_CONFIG_STORAGE_VOL, - "doc", xml, - "schema", DATADIR "/libvirt/schemas/storage_vol.rng", - NULL)); + "node", doc->children, + "schema", DATADIR "/libvirt/schemas/storage_vol.rng", + NULL)); } diff --git a/libvirt-gconfig/libvirt-gconfig-storage-vol.h b/libvirt-gconfig/libvirt-gconfig-storage-vol.h index 73032b6..08a2180 100644 --- a/libvirt-gconfig/libvirt-gconfig-storage-vol.h +++ b/libvirt-gconfig/libvirt-gconfig-storage-vol.h @@ -59,7 +59,7 @@ struct _GVirConfigStorageVolClass GType gvir_config_storage_vol_get_type(void); -GVirConfigStorageVol *gvir_config_storage_vol_new(const gchar *xml); +GVirConfigStorageVol *gvir_config_storage_vol_new(void); G_END_DECLS diff --git a/libvirt-gobject/libvirt-gobject-domain-snapshot.c b/libvirt-gobject/libvirt-gobject-domain-snapshot.c index cc17b60..530907d 100644 --- a/libvirt-gobject/libvirt-gobject-domain-snapshot.c +++ b/libvirt-gobject/libvirt-gobject-domain-snapshot.c @@ -213,8 +213,11 @@ GVirConfigDomainSnapshot *gvir_domain_snapshot_get_config return NULL; } +#if 0 GVirConfigDomainSnapshot *conf = gvir_config_domain_snapshot_new(xml); g_free(xml); return conf; +#endif + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-interface.c b/libvirt-gobject/libvirt-gobject-interface.c index b37a7fa..2c56492 100644 --- a/libvirt-gobject/libvirt-gobject-interface.c +++ b/libvirt-gobject/libvirt-gobject-interface.c @@ -203,8 +203,11 @@ GVirConfigInterface *gvir_interface_get_config(GVirInterface *iface, return NULL; } +#if 0 GVirConfigInterface *conf = gvir_config_interface_new(xml); g_free(xml); return conf; +#endif + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-network-filter.c b/libvirt-gobject/libvirt-gobject-network-filter.c index eb10ce9..b043e42 100644 --- a/libvirt-gobject/libvirt-gobject-network-filter.c +++ b/libvirt-gobject/libvirt-gobject-network-filter.c @@ -228,8 +228,11 @@ GVirConfigNetworkFilter *gvir_network_filter_get_config return NULL; } +#if 0 GVirConfigNetworkFilter *conf = gvir_config_network_filter_new(xml); g_free(xml); return conf; +#endif + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-network.c b/libvirt-gobject/libvirt-gobject-network.c index 158afa4..34692fd 100644 --- a/libvirt-gobject/libvirt-gobject-network.c +++ b/libvirt-gobject/libvirt-gobject-network.c @@ -224,8 +224,11 @@ GVirConfigNetwork *gvir_network_get_config(GVirNetwork *network, return NULL; } +#if 0 GVirConfigNetwork *conf = gvir_config_network_new(xml); g_free(xml); return conf; +#endif + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-node-device.c b/libvirt-gobject/libvirt-gobject-node-device.c index 2847fc6..b3ae6bf 100644 --- a/libvirt-gobject/libvirt-gobject-node-device.c +++ b/libvirt-gobject/libvirt-gobject-node-device.c @@ -203,8 +203,12 @@ GVirConfigNodeDevice *gvir_node_device_get_config(GVirNodeDevice *device, return NULL; } +#if 0 GVirConfigNodeDevice *conf = gvir_config_node_device_new(xml); g_free(xml); return conf; +#endif + + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-secret.c b/libvirt-gobject/libvirt-gobject-secret.c index ffff521..b39bac5 100644 --- a/libvirt-gobject/libvirt-gobject-secret.c +++ b/libvirt-gobject/libvirt-gobject-secret.c @@ -214,8 +214,12 @@ GVirConfigSecret *gvir_secret_get_config(GVirSecret *secret, return NULL; } +#if 0 GVirConfigSecret *conf = gvir_config_secret_new(xml); g_free(xml); return conf; +#endif + + return NULL; } diff --git a/libvirt-gobject/libvirt-gobject-storage-pool.c b/libvirt-gobject/libvirt-gobject-storage-pool.c index 8cc481e..d790a92 100644 --- a/libvirt-gobject/libvirt-gobject-storage-pool.c +++ b/libvirt-gobject/libvirt-gobject-storage-pool.c @@ -239,10 +239,13 @@ GVirConfigStoragePool *gvir_storage_pool_get_config(GVirStoragePool *pool, return NULL; } +#if 0 GVirConfigStoragePool *conf = gvir_config_storage_pool_new(xml); g_free(xml); return conf; +#endif + return NULL; } typedef gint (* CountFunction) (virStoragePoolPtr vpool); diff --git a/libvirt-gobject/libvirt-gobject-storage-vol.c b/libvirt-gobject/libvirt-gobject-storage-vol.c index 6cbae84..0d94cd5 100644 --- a/libvirt-gobject/libvirt-gobject-storage-vol.c +++ b/libvirt-gobject/libvirt-gobject-storage-vol.c @@ -214,8 +214,11 @@ GVirConfigStorageVol *gvir_storage_vol_get_config(GVirStorageVol *vol, return NULL; } +#if 0 GVirConfigStorageVol *conf = gvir_config_storage_vol_new(xml); g_free(xml); return conf; +#endif + return NULL; }

This commit changes gvir_config_domain_new_from_xml not to operate on an existing object. This makes it possible to call it to create an appropriate xmlNodePtr object which can then be used to construct an object derived from GVirConfigObject. This will also makes it possible to remove GVirConfigObject::doc in a subsequent commit. --- libvirt-gconfig/libvirt-gconfig-domain.c | 16 +++++----- libvirt-gconfig/libvirt-gconfig-domain.h | 2 +- libvirt-gconfig/libvirt-gconfig-object.c | 41 +++++----------------------- libvirt-gconfig/libvirt-gconfig-object.h | 3 ++ libvirt-gconfig/tests/test-domain-parse.c | 6 +++- libvirt-gobject/libvirt-gobject-domain.c | 2 +- 6 files changed, 26 insertions(+), 44 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 00cab80..ffd707d 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -112,10 +112,16 @@ static void gvir_config_domain_init(GVirConfigDomain *conn) } -GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml) +GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml, + GError **error) { + xmlNodePtr node; + + node = gvir_config_xml_parse(xml, "domain", error); + if ((error != NULL) && (*error != NULL)) + return NULL; return GVIR_CONFIG_DOMAIN(g_object_new(GVIR_TYPE_CONFIG_DOMAIN, - "doc", xml, + "node", node, "schema", DATADIR "/libvirt/schemas/domain.rng", NULL)); } @@ -132,10 +138,6 @@ GVirConfigDomain *gvir_config_domain_new(void) NULL)); } -/* FIXME: do we add a GError ** to all getters in case there's an XML - * parsing error? Doesn't work with gobject properties - * => have a function to test if an error has occurred a la cairo? - */ char *gvir_config_domain_get_name(GVirConfigDomain *domain) { xmlNodePtr node; @@ -145,7 +147,6 @@ char *gvir_config_domain_get_name(GVirConfigDomain *domain) return NULL; return gvir_config_xml_get_child_element_content_glib(node, "name"); - } void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) @@ -164,7 +165,6 @@ void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) xmlFree(encoded_name); old_node = gvir_config_xml_get_element(parent_node, "name", NULL); - if (old_node) { old_node = xmlReplaceNode(old_node, new_node); xmlFreeNode(old_node); diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h b/libvirt-gconfig/libvirt-gconfig-domain.h index f6ceef1..b5ae050 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.h +++ b/libvirt-gconfig/libvirt-gconfig-domain.h @@ -59,7 +59,7 @@ struct _GVirConfigDomainClass GType gvir_config_domain_get_type(void); -GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml); +GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml, GError **error); GVirConfigDomain *gvir_config_domain_new(void); char *gvir_config_domain_get_name(GVirConfigDomain *domain); diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index b7829c9..bcb622a 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -26,10 +26,10 @@ #include <string.h> #include <libxml/relaxng.h> -#include <libxml/xmlerror.h> #include "libvirt-gconfig/libvirt-gconfig.h" + //extern gboolean debugFlag; gboolean debugFlag; @@ -67,6 +67,7 @@ static void gvir_xml_structured_error_nop(void *userData G_GNUC_UNUSED, { } + static void gvir_config_object_get_property(GObject *object, guint prop_id, GValue *value, @@ -93,7 +94,6 @@ static void gvir_config_object_get_property(GObject *object, } } - static void gvir_config_object_set_property(GObject *object, guint prop_id, const GValue *value, @@ -209,34 +209,6 @@ static void gvir_config_object_init(GVirConfigObject *conn) memset(priv, 0, sizeof(*priv)); } -static void -gvir_config_object_parse(GVirConfigObject *config, - GError **err) -{ - GVirConfigObjectPrivate *priv = config->priv; - xmlDocPtr doc; - if (priv->node) - return; - - if (!priv->doc) { - *err = g_error_new(GVIR_CONFIG_OBJECT_ERROR, - 0, - "%s", - "No XML document to parse"); - return; - } - - doc = xmlParseMemory(priv->doc, strlen(priv->doc)); - if (!doc) { - *err = gvir_xml_error_new(GVIR_CONFIG_OBJECT_ERROR, - 0, - "%s", - "Unable to parse configuration"); - } - priv->node = doc->children; -} - - void gvir_config_object_validate(GVirConfigObject *config, GError **err) { @@ -248,9 +220,13 @@ void gvir_config_object_validate(GVirConfigObject *config, xmlSetGenericErrorFunc(NULL, gvir_xml_generic_error_nop); xmlSetStructuredErrorFunc(NULL, gvir_xml_structured_error_nop); - gvir_config_object_parse(config, err); - if (*err) + if (!priv->node) { + *err = gvir_xml_error_new(GVIR_CONFIG_OBJECT_ERROR, + 0, + "%s", + "No XML document associated with this config object"); return; + } rngParser = xmlRelaxNGNewParserCtxt(priv->schema); if (!rngParser) { @@ -333,6 +309,5 @@ const gchar *gvir_config_object_get_schema(GVirConfigObject *config) xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config, GError **error) { - gvir_config_object_parse(config, error); return config->priv->node; } diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index 40480ba..98a05cb 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -68,6 +68,9 @@ 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, GError **error); +/* FIXME: move to a libvirt-gconfig-helpers.h file? */ +xmlNodePtr gvir_config_object_parse(const char *xml, const char *root_node, GError **err); + G_END_DECLS #endif /* __LIBVIRT_GCONFIG_OBJECT_H__ */ diff --git a/libvirt-gconfig/tests/test-domain-parse.c b/libvirt-gconfig/tests/test-domain-parse.c index 3a36144..57b1875 100644 --- a/libvirt-gconfig/tests/test-domain-parse.c +++ b/libvirt-gconfig/tests/test-domain-parse.c @@ -50,7 +50,11 @@ int main(int argc, char **argv) g_type_init(); - domain = gvir_config_domain_new_from_xml(xml); + domain = gvir_config_domain_new_from_xml(xml, &error); + if (error != NULL) { + g_print("Couldn't parse %s: %s\n", argv[1], error->message); + return 3; + } g_assert(domain != NULL); name = gvir_config_domain_get_name(domain); g_assert(name != NULL); diff --git a/libvirt-gobject/libvirt-gobject-domain.c b/libvirt-gobject/libvirt-gobject-domain.c index fd5f709..c5df290 100644 --- a/libvirt-gobject/libvirt-gobject-domain.c +++ b/libvirt-gobject/libvirt-gobject-domain.c @@ -432,7 +432,7 @@ GVirConfigDomain *gvir_domain_get_config(GVirDomain *dom, return NULL; } - GVirConfigDomain *conf = gvir_config_domain_new_from_xml(xml); + GVirConfigDomain *conf = gvir_config_domain_new_from_xml(xml, err); g_free(xml); if ((err != NULL) && (*err != NULL)) return NULL;

Since this function no longer tries to do XML parsing, there is no error to report. --- libvirt-gconfig/libvirt-gconfig-domain.c | 5 ++--- libvirt-gconfig/libvirt-gconfig-object.c | 7 +++---- libvirt-gconfig/libvirt-gconfig-object.h | 2 +- 3 files changed, 6 insertions(+), 8 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index ffd707d..210b3fa 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -142,7 +142,7 @@ char *gvir_config_domain_get_name(GVirConfigDomain *domain) { xmlNodePtr node; - node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain), NULL); + node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); if (node == NULL) return NULL; @@ -156,8 +156,7 @@ void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) xmlNodePtr new_node; xmlChar *encoded_name; - parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain), - NULL); + parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); encoded_name = xmlEncodeEntitiesReentrant(parent_node->doc, (xmlChar *)name); new_node = xmlNewDocNode(parent_node->doc, NULL, diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index bcb622a..97ac2e3 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -86,7 +86,7 @@ static void gvir_config_object_get_property(GObject *object, break; case PROP_NODE: - g_value_set_pointer(value, gvir_config_object_get_xml_node(conn, NULL)); + g_value_set_pointer(value, gvir_config_object_get_xml_node(conn)); break; default: @@ -279,7 +279,7 @@ gchar *gvir_config_object_to_xml(GVirConfigObject *config) xmlNodePtr node; gchar *output_doc; - node = gvir_config_object_get_xml_node(config, NULL); + node = gvir_config_object_get_xml_node(config); if (node == NULL) return NULL; @@ -306,8 +306,7 @@ const gchar *gvir_config_object_get_schema(GVirConfigObject *config) /* FIXME: will we always have one xmlNode per GConfig object? */ /* FIXME: need to return the right node from subclasses */ /* NB: the xmlNodePtr must not be freed by the caller */ -xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config, - GError **error) +xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config) { return config->priv->node; } diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index 98a05cb..5860102 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -66,7 +66,7 @@ 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, GError **error); +xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config); /* FIXME: move to a libvirt-gconfig-helpers.h file? */ xmlNodePtr gvir_config_object_parse(const char *xml, const char *root_node, GError **err);

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;

There will be quite a few getter functions returning a string contained in one of the nodes of a GVirConfigObject, so better to make it a one-liner to achieve this. --- libvirt-gconfig/libvirt-gconfig-domain.c | 9 ++------- libvirt-gconfig/libvirt-gconfig-object.c | 12 ++++++++++++ libvirt-gconfig/libvirt-gconfig-object.h | 2 ++ 3 files changed, 16 insertions(+), 7 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 210b3fa..66ea7cb 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -140,13 +140,8 @@ GVirConfigDomain *gvir_config_domain_new(void) char *gvir_config_domain_get_name(GVirConfigDomain *domain) { - xmlNodePtr node; - - node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); - if (node == NULL) - return NULL; - - return gvir_config_xml_get_child_element_content_glib(node, "name"); + return gvir_config_object_get_node_content(GVIR_CONFIG_OBJECT(domain), + "name"); } void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index 6fd5473..8855ff8 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -280,3 +280,15 @@ xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config) { return config->priv->node; } + +char *gvir_config_object_get_node_content(GVirConfigObject *object, + const char *node_name) +{ + xmlNodePtr node; + + node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(object)); + if (node == NULL) + return NULL; + + return gvir_config_xml_get_child_element_content_glib(node, node_name); +} diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index 133ca9f..1a82490 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -66,6 +66,8 @@ gchar *gvir_config_object_to_xml(GVirConfigObject *config); const gchar *gvir_config_object_get_schema(GVirConfigObject *config); xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config); +char *gvir_config_object_get_node_content(GVirConfigObject *object, + const char *node_name); /* FIXME: move to a libvirt-gconfig-helpers.h file? */ xmlNodePtr gvir_config_object_parse(const char *xml, const char *root_node, GError **err);

It's used to set the content of a node in a GVirConfigObject to a given value, and will replace any existing node with the same value. --- libvirt-gconfig/libvirt-gconfig-domain.c | 22 ++-------------------- libvirt-gconfig/libvirt-gconfig-object.c | 30 ++++++++++++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-object.h | 3 +++ 3 files changed, 35 insertions(+), 20 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 66ea7cb..039e551 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -146,25 +146,7 @@ char *gvir_config_domain_get_name(GVirConfigDomain *domain) void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) { - xmlNodePtr parent_node; - xmlNodePtr old_node; - xmlNodePtr new_node; - xmlChar *encoded_name; - - parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); - encoded_name = xmlEncodeEntitiesReentrant(parent_node->doc, - (xmlChar *)name); - new_node = xmlNewDocNode(parent_node->doc, NULL, - (xmlChar *)"name", encoded_name); - xmlFree(encoded_name); - - old_node = gvir_config_xml_get_element(parent_node, "name", NULL); - if (old_node) { - old_node = xmlReplaceNode(old_node, new_node); - xmlFreeNode(old_node); - } else { - xmlAddChild(parent_node, new_node); - } - + gvir_config_object_set_node_content(GVIR_CONFIG_OBJECT(domain), + "name", name); g_object_notify(G_OBJECT(domain), "name"); } diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index 8855ff8..679c3f9 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -292,3 +292,33 @@ char *gvir_config_object_get_node_content(GVirConfigObject *object, return gvir_config_xml_get_child_element_content_glib(node, node_name); } + +/* FIXME: if there are multiple nodes with the same name, this function + * won't behave as expected. Should we get rid of the duplicated node names + * here? + */ +void gvir_config_object_set_node_content(GVirConfigObject *object, + const char *node_name, + const char *value) +{ + xmlNodePtr parent_node; + xmlNodePtr old_node; + xmlNodePtr new_node; + xmlChar *encoded_name; + + parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(object)); + encoded_name = xmlEncodeEntitiesReentrant(parent_node->doc, + (xmlChar *)value); + new_node = xmlNewDocNode(parent_node->doc, NULL, + (xmlChar *)node_name, encoded_name); + xmlFree(encoded_name); + + old_node = gvir_config_xml_get_element(parent_node, node_name, NULL); + if (old_node) { + old_node = xmlReplaceNode(old_node, new_node); + xmlFreeNode(old_node); + } else { + xmlAddChild(parent_node, new_node); + } +} + diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index 1a82490..ab99279 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -68,6 +68,9 @@ const gchar *gvir_config_object_get_schema(GVirConfigObject *config); xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config); char *gvir_config_object_get_node_content(GVirConfigObject *object, const char *node_name); +void gvir_config_object_set_node_content(GVirConfigObject *object, + const char *node_name, + const char *value); /* FIXME: move to a libvirt-gconfig-helpers.h file? */ xmlNodePtr gvir_config_object_parse(const char *xml, const char *root_node, GError **err);

This test has 3 duplicate "name" nodes in the domain XML, I don't know what the right behaviour is in this case :)

--- libvirt-gconfig/tests/test-domain-duplicate.xml | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-) create mode 100644 libvirt-gconfig/tests/test-domain-duplicate.xml diff --git a/libvirt-gconfig/tests/test-domain-duplicate.xml b/libvirt-gconfig/tests/test-domain-duplicate.xml new file mode 100644 index 0000000..acff40a --- /dev/null +++ b/libvirt-gconfig/tests/test-domain-duplicate.xml @@ -0,0 +1,7 @@ +<domain type='xen' id='3'> + <name>bar</name> + <name>foo</name> + <name>foobar</name> + <uuid>4dea22b31d52d8f32516782e98ab3fa0</uuid> + <description>Some human readable description</description> +</domain>

--- libvirt-gconfig/libvirt-gconfig-domain.c | 29 +++++++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-domain.h | 2 + libvirt-gconfig/libvirt-gconfig-object.c | 32 ++++++++++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-object.h | 5 ++++ libvirt-gconfig/libvirt-gconfig.sym | 2 + libvirt-gconfig/tests/test-domain-create.c | 3 ++ libvirt-gconfig/tests/test-domain-parse.c | 3 ++ libvirt-gconfig/tests/test-domain.xml | 1 + 8 files changed, 77 insertions(+), 0 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 039e551..71bfb55 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -44,6 +44,7 @@ G_DEFINE_TYPE(GVirConfigDomain, gvir_config_domain, GVIR_TYPE_CONFIG_OBJECT); enum { PROP_0, PROP_NAME, + PROP_MEMORY, }; static void gvir_config_domain_get_property(GObject *object, @@ -57,6 +58,9 @@ static void gvir_config_domain_get_property(GObject *object, case PROP_NAME: g_value_take_string(value, gvir_config_domain_get_name(domain)); break; + case PROP_MEMORY: + g_value_set_uint64(value, gvir_config_domain_get_memory(domain)); + break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec); @@ -74,6 +78,9 @@ static void gvir_config_domain_set_property(GObject *object, case PROP_NAME: gvir_config_domain_set_name(domain, g_value_get_string(value)); break; + case PROP_MEMORY: + gvir_config_domain_set_memory(domain, g_value_get_uint64(value)); + break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec); } @@ -97,6 +104,15 @@ static void gvir_config_domain_class_init(GVirConfigDomainClass *klass) NULL, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); + g_object_class_install_property(object_class, + PROP_MEMORY, + g_param_spec_uint64("memory", + "Memory", + "Maximum Guest Memory (in kilobytes)", + 0, G_MAXUINT64, + 0, + G_PARAM_READWRITE | + G_PARAM_STATIC_STRINGS)); } @@ -150,3 +166,16 @@ void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name) "name", name); g_object_notify(G_OBJECT(domain), "name"); } + +guint64 gvir_config_domain_get_memory(GVirConfigDomain *domain) +{ + return gvir_config_object_get_node_content_uint64(GVIR_CONFIG_OBJECT(domain), + "memory"); +} + +void gvir_config_domain_set_memory(GVirConfigDomain *domain, guint64 memory) +{ + gvir_config_object_set_node_content_uint64(GVIR_CONFIG_OBJECT(domain), + "memory", memory); + g_object_notify(G_OBJECT(domain), "memory"); +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h b/libvirt-gconfig/libvirt-gconfig-domain.h index b5ae050..03169b2 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.h +++ b/libvirt-gconfig/libvirt-gconfig-domain.h @@ -64,6 +64,8 @@ GVirConfigDomain *gvir_config_domain_new(void); char *gvir_config_domain_get_name(GVirConfigDomain *domain); void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name); +guint64 gvir_config_domain_get_memory(GVirConfigDomain *domain); +void gvir_config_domain_set_memory(GVirConfigDomain *domain, guint64 memory); G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig-object.c b/libvirt-gconfig/libvirt-gconfig-object.c index 679c3f9..0de4abf 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.c +++ b/libvirt-gconfig/libvirt-gconfig-object.c @@ -322,3 +322,35 @@ void gvir_config_object_set_node_content(GVirConfigObject *object, } } +/* FIXME: how to notify of errors/node not found? */

+guint64 gvir_config_object_get_node_content_uint64(GVirConfigObject *object, + const char *node_name) +{ + xmlNodePtr node; + xmlChar *str; + guint64 value; + + node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(object)); + if (node == NULL) + return 0; + + str = gvir_config_xml_get_child_element_content(node, node_name); + if (!str) + return 0; + + value = g_ascii_strtoull((char *)str, NULL, 0); + xmlFree(str); + + return value; +}

+ + +void gvir_config_object_set_node_content_uint64(GVirConfigObject *object, + const char *node_name, + guint64 value) +{ + char *str; + str = g_strdup_printf("%"G_GUINT64_FORMAT, value); + gvir_config_object_set_node_content(object, node_name, str); + g_free(str); +} diff --git a/libvirt-gconfig/libvirt-gconfig-object.h b/libvirt-gconfig/libvirt-gconfig-object.h index ab99279..c203798 100644 --- a/libvirt-gconfig/libvirt-gconfig-object.h +++ b/libvirt-gconfig/libvirt-gconfig-object.h @@ -68,9 +68,14 @@ const gchar *gvir_config_object_get_schema(GVirConfigObject *config); xmlNodePtr gvir_config_object_get_xml_node(GVirConfigObject *config); char *gvir_config_object_get_node_content(GVirConfigObject *object, const char *node_name); +guint64 gvir_config_object_get_node_content_uint64(GVirConfigObject *object, + const char *node_name); void gvir_config_object_set_node_content(GVirConfigObject *object, const char *node_name, const char *value); +void gvir_config_object_set_node_content_uint64(GVirConfigObject *object, + const char *node_name, + guint64 value); /* FIXME: move to a libvirt-gconfig-helpers.h file? */ xmlNodePtr gvir_config_object_parse(const char *xml, const char *root_node, GError **err); diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym index b67d152..7acb95d 100644 --- a/libvirt-gconfig/libvirt-gconfig.sym +++ b/libvirt-gconfig/libvirt-gconfig.sym @@ -6,6 +6,8 @@ LIBVIRT_GOBJECT_0.0.1 { gvir_config_domain_get_type; gvir_config_domain_new; gvir_config_domain_new_from_xml; + gvir_config_domain_get_memory; + gvir_config_domain_set_memory; gvir_config_domain_get_name; gvir_config_domain_set_name; diff --git a/libvirt-gconfig/tests/test-domain-create.c b/libvirt-gconfig/tests/test-domain-create.c index a742dd7..fe0063c 100644 --- a/libvirt-gconfig/tests/test-domain-create.c +++ b/libvirt-gconfig/tests/test-domain-create.c @@ -44,6 +44,9 @@ int main(void) g_assert(strcmp(name, "foo") == 0); g_free(name); + gvir_config_domain_set_memory(domain, 1234); + g_assert(gvir_config_domain_get_memory(domain) == 1234); + xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(domain)); g_print("%s\n", xml); g_free(xml); diff --git a/libvirt-gconfig/tests/test-domain-parse.c b/libvirt-gconfig/tests/test-domain-parse.c index 545e733..7521d20 100644 --- a/libvirt-gconfig/tests/test-domain-parse.c +++ b/libvirt-gconfig/tests/test-domain-parse.c @@ -66,6 +66,9 @@ int main(int argc, char **argv) g_assert(name != NULL); g_assert(strcmp(name, "foo") == 0); g_free(name); + + g_assert(gvir_config_domain_get_memory(domain) == 987654321); + g_free(xml); xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(domain)); diff --git a/libvirt-gconfig/tests/test-domain.xml b/libvirt-gconfig/tests/test-domain.xml index 08a926e..d887e95 100644 --- a/libvirt-gconfig/tests/test-domain.xml +++ b/libvirt-gconfig/tests/test-domain.xml @@ -2,4 +2,5 @@ <name>foo</name> <uuid>4dea22b31d52d8f32516782e98ab3fa0</uuid> <description>Some human readable description</description> + <memory>987654321</memory> </domain>

--- libvirt-gconfig/libvirt-gconfig-domain.c | 71 ++++++++++++++++++++++++++++ libvirt-gconfig/libvirt-gconfig-domain.h | 4 ++ libvirt-gconfig/libvirt-gconfig.sym | 2 + libvirt-gconfig/tests/test-domain-create.c | 12 +++++ libvirt-gconfig/tests/test-domain-parse.c | 7 +++ libvirt-gconfig/tests/test-domain.xml | 1 + 6 files changed, 97 insertions(+), 0 deletions(-) diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c b/libvirt-gconfig/libvirt-gconfig-domain.c index 71bfb55..4a32595 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.c +++ b/libvirt-gconfig/libvirt-gconfig-domain.c @@ -45,6 +45,7 @@ enum { PROP_0, PROP_NAME, PROP_MEMORY, + PROP_FEATURES }; static void gvir_config_domain_get_property(GObject *object, @@ -61,6 +62,9 @@ static void gvir_config_domain_get_property(GObject *object, case PROP_MEMORY: g_value_set_uint64(value, gvir_config_domain_get_memory(domain)); break; + case PROP_FEATURES: + g_value_take_boxed(value, gvir_config_domain_get_features(domain)); + break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec); @@ -81,6 +85,9 @@ static void gvir_config_domain_set_property(GObject *object, case PROP_MEMORY: gvir_config_domain_set_memory(domain, g_value_get_uint64(value)); break; + case PROP_FEATURES: + gvir_config_domain_set_features(domain, g_value_get_boxed(value)); + break; default: G_OBJECT_WARN_INVALID_PROPERTY_ID(object, prop_id, pspec); } @@ -113,6 +120,14 @@ static void gvir_config_domain_class_init(GVirConfigDomainClass *klass) 0, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS)); + g_object_class_install_property(object_class, + PROP_FEATURES, + g_param_spec_boxed("features", + "Features", + "Hypervisor Features", + G_TYPE_STRV, + G_PARAM_READWRITE | + G_PARAM_STATIC_STRINGS)); } @@ -179,3 +194,59 @@ void gvir_config_domain_set_memory(GVirConfigDomain *domain, guint64 memory) "memory", memory); g_object_notify(G_OBJECT(domain), "memory"); } + +/** + * gvir_config_domain_get_features: + * Returns: (transfer full): + */ +GStrv gvir_config_domain_get_features(GVirConfigDomain *domain) +{ + GPtrArray *features; + xmlNodePtr parent_node; + xmlNodePtr node; + xmlNodePtr it; + + parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); + if (parent_node == NULL) + return NULL; + + node = gvir_config_xml_get_element(parent_node, "features", NULL); + if (node == NULL) + return NULL; + + features = g_ptr_array_new(); + for (it = node->children; it != NULL; it = it->next) { + g_ptr_array_add(features, g_strdup((char *)it->name)); + } + g_ptr_array_add(features, NULL); + + return (GStrv)g_ptr_array_free(features, FALSE); +} + +void gvir_config_domain_set_features(GVirConfigDomain *domain, + const GStrv features) +{ + xmlNodePtr parent_node; + xmlNodePtr features_node; + xmlNodePtr old_node; + GStrv it; + + parent_node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(domain)); + features_node = xmlNewDocNode(parent_node->doc, NULL, + (xmlChar *)"features", NULL); + for (it = features; *it != NULL; it++) { + xmlNodePtr node; + + node = xmlNewDocNode(parent_node->doc, NULL, (xmlChar *)*it, NULL); + xmlAddChild(features_node, node); + } + + old_node = gvir_config_xml_get_element(parent_node, "features", NULL); + if (old_node) { + old_node = xmlReplaceNode(old_node, features_node); + xmlFreeNode(old_node); + } else { + xmlAddChild(parent_node, features_node); + } + g_object_notify(G_OBJECT(domain), "features"); +} diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h b/libvirt-gconfig/libvirt-gconfig-domain.h index 03169b2..d9f0c09 100644 --- a/libvirt-gconfig/libvirt-gconfig-domain.h +++ b/libvirt-gconfig/libvirt-gconfig-domain.h @@ -66,6 +66,10 @@ char *gvir_config_domain_get_name(GVirConfigDomain *domain); void gvir_config_domain_set_name(GVirConfigDomain *domain, const char *name); guint64 gvir_config_domain_get_memory(GVirConfigDomain *domain); void gvir_config_domain_set_memory(GVirConfigDomain *domain, guint64 memory); +GStrv gvir_config_domain_get_features(GVirConfigDomain *domain); +void gvir_config_domain_set_features(GVirConfigDomain *domain, + const GStrv features); + G_END_DECLS diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym index 7acb95d..951aac6 100644 --- a/libvirt-gconfig/libvirt-gconfig.sym +++ b/libvirt-gconfig/libvirt-gconfig.sym @@ -6,6 +6,8 @@ LIBVIRT_GOBJECT_0.0.1 { gvir_config_domain_get_type; gvir_config_domain_new; gvir_config_domain_new_from_xml; + gvir_config_domain_get_features; + gvir_config_domain_set_features; gvir_config_domain_get_memory; gvir_config_domain_set_memory; gvir_config_domain_get_name; diff --git a/libvirt-gconfig/tests/test-domain-create.c b/libvirt-gconfig/tests/test-domain-create.c index fe0063c..a719ed2 100644 --- a/libvirt-gconfig/tests/test-domain-create.c +++ b/libvirt-gconfig/tests/test-domain-create.c @@ -28,10 +28,14 @@ #include <string.h> #include <libvirt-gconfig/libvirt-gconfig.h> +const char *features[] = { "foo", "bar", "baz", NULL }; + int main(void) { GVirConfigDomain *domain; char *name; + GStrv feat; + unsigned int i; char *xml; g_type_init(); @@ -47,6 +51,14 @@ int main(void) gvir_config_domain_set_memory(domain, 1234); g_assert(gvir_config_domain_get_memory(domain) == 1234); + gvir_config_domain_set_features(domain, (const GStrv)features); + feat = gvir_config_domain_get_features(domain); + for (i = 0; features[i] != NULL; i++) { + g_assert(feat[i] != NULL); + g_assert(strcmp(feat[i], features[i]) == 0); + } + g_strfreev(feat); + xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(domain)); g_print("%s\n", xml); g_free(xml); diff --git a/libvirt-gconfig/tests/test-domain-parse.c b/libvirt-gconfig/tests/test-domain-parse.c index 7521d20..9528816 100644 --- a/libvirt-gconfig/tests/test-domain-parse.c +++ b/libvirt-gconfig/tests/test-domain-parse.c @@ -33,6 +33,7 @@ int main(int argc, char **argv) { GVirConfigDomain *domain; char *name; + GStrv features; char *xml; GError *error = NULL; @@ -69,6 +70,12 @@ int main(int argc, char **argv) g_assert(gvir_config_domain_get_memory(domain) == 987654321); + features = gvir_config_domain_get_features(domain); + g_assert(g_strv_length(features) == 2); + g_assert(strcmp(features[0], "f1") == 0); + g_assert(strcmp(features[1], "f2") == 0); + g_strfreev(features); + g_free(xml); xml = gvir_config_object_to_xml(GVIR_CONFIG_OBJECT(domain)); diff --git a/libvirt-gconfig/tests/test-domain.xml b/libvirt-gconfig/tests/test-domain.xml index d887e95..a9610cc 100644 --- a/libvirt-gconfig/tests/test-domain.xml +++ b/libvirt-gconfig/tests/test-domain.xml @@ -2,5 +2,6 @@ <name>foo</name> <uuid>4dea22b31d52d8f32516782e98ab3fa0</uuid> <description>Some human readable description</description> + <features><f1/><f2/></features> <memory>987654321</memory> </domain>