On 05/18/2011 11:47 AM, Michal Privoznik wrote: > --- > src/conf/cpu_conf.c | 7 ++++--- > src/conf/domain_conf.c | 12 ++++++++---- > src/conf/interface_conf.c | 6 ++++-- > src/conf/network_conf.c | 6 ++++-- > src/conf/node_device_conf.c | 6 ++++-- > src/conf/secret_conf.c | 6 ++++-- > src/conf/storage_conf.c | 12 ++++++++---- > src/security/virt-aa-helper.c | 3 ++- > src/test/test_driver.c | 13 ++++++++++--- > src/util/xml.c | 2 +- > 10 files changed, 49 insertions(+), 24 deletions(-) > I think most of these error messages as well can be consolidated in virXMLParseFile. The caller can pass in the root element name, and virXMLParseFile can just do all this validation, and return a XPathContext pointer. If any caller needs the xmlDocPtr, it's stored in ctxt->doc. Thanks, Cole -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list