On Fri, Oct 07, 2011 at 11:40:51AM +0200, Christophe Fergeau wrote:
Rename it to gvir_config_domain_new_from_xml since it takes an
XML string as input. gvir_config_domain_new will be used to create
a new empty domain.
---
libvirt-gconfig/libvirt-gconfig-domain.c | 2 +-
libvirt-gconfig/libvirt-gconfig-domain.h | 2 +-
libvirt-gconfig/libvirt-gconfig.sym | 2 +-
libvirt-gobject/libvirt-gobject-domain.c | 6 ++++--
4 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/libvirt-gconfig/libvirt-gconfig-domain.c
b/libvirt-gconfig/libvirt-gconfig-domain.c
index 125c1ac..2b9419e 100644
--- a/libvirt-gconfig/libvirt-gconfig-domain.c
+++ b/libvirt-gconfig/libvirt-gconfig-domain.c
@@ -95,7 +95,7 @@ static void gvir_config_domain_init(GVirConfigDomain *conn)
}
-GVirConfigDomain *gvir_config_domain_new(const gchar *xml)
+GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml)
{
return GVIR_CONFIG_DOMAIN(g_object_new(GVIR_TYPE_CONFIG_DOMAIN,
"doc", xml,
diff --git a/libvirt-gconfig/libvirt-gconfig-domain.h
b/libvirt-gconfig/libvirt-gconfig-domain.h
index 49c34dc..343f076 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(const gchar *xml);
+GVirConfigDomain *gvir_config_domain_new_from_xml(const gchar *xml);
char *gvir_config_domain_get_name(GVirConfigDomain *domain);
diff --git a/libvirt-gconfig/libvirt-gconfig.sym b/libvirt-gconfig/libvirt-gconfig.sym
index 6127e22..76e2ba2 100644
--- a/libvirt-gconfig/libvirt-gconfig.sym
+++ b/libvirt-gconfig/libvirt-gconfig.sym
@@ -4,7 +4,7 @@ LIBVIRT_GOBJECT_0.0.1 {
gvir_config_capabilities_new;
gvir_config_domain_get_type;
- gvir_config_domain_new;
+ gvir_config_domain_new_from_xml;
gvir_config_domain_get_name;
gvir_config_domain_snapshot_get_type;
diff --git a/libvirt-gobject/libvirt-gobject-domain.c
b/libvirt-gobject/libvirt-gobject-domain.c
index 70d8d6d..fd5f709 100644
--- a/libvirt-gobject/libvirt-gobject-domain.c
+++ b/libvirt-gobject/libvirt-gobject-domain.c
@@ -432,9 +432,11 @@ GVirConfigDomain *gvir_domain_get_config(GVirDomain *dom,
return NULL;
}
- GVirConfigDomain *conf = gvir_config_domain_new(xml);
-
+ GVirConfigDomain *conf = gvir_config_domain_new_from_xml(xml);
g_free(xml);
+ if ((err != NULL) && (*err != NULL))
+ return NULL;
+
return conf;
}
ACK
Daniel
--
|:
http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|:
http://entangle-photo.org -o-
http://live.gnome.org/gtk-vnc :|