On Fri, Oct 16, 2009 at 12:35:11PM -0400, Cole Robinson wrote: > We need to parse a source XML block for FindPoolSources, so this is a step > in sharing the parsing. The new storage pool XML 2 XML tests cover this area > pretty well to ensure we aren't causing regressions. > > Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx> Refactoring looks fine to me, it seems changing the context node is fine as it's properly restaured and XPath expressions are changed to be made relative to the new node. ACK, thanks ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list