Re: [libvirt] [PATCH 3/4] Improve incorrect root element error messages

[Cole Robinson <crobinso redhat com>]

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