Refactor few old-style XML node lookup loops

[PATCH 0/5] Refactor few old-style XML node lookup loops

Peter Krempa posted 5 patches 8 months, 1 week ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1693231019.git.pkrempa@redhat.com

src/conf/domain_conf.c  | 50 +++++++++++-----------
src/conf/network_conf.c | 94 ++++++++++++++++++++---------------------
tools/virsh-domain.c    | 63 +++------------------------
3 files changed, 78 insertions(+), 129 deletions(-)

Expand all Fold all

[PATCH 0/5] Refactor few old-style XML node lookup loops

Posted by Peter Krempa 8 months, 1 week ago

Peter Krempa (5):
  virNetworkDNSHostDefParseXML: Refactor parsing
  virsh: domain: Refactor XML handling for disk changes
  virDomainFeaturesCapabilitiesDefParse: Use virXMLNodeGetSubelementList
  virDomainFeaturesKVMDefParse: Use virXMLNodeGetSubelementList
  virDomainFeaturesXENDefParse: Use virXMLNodeGetSubelementList

 src/conf/domain_conf.c  | 50 +++++++++++-----------
 src/conf/network_conf.c | 94 ++++++++++++++++++++---------------------
 tools/virsh-domain.c    | 63 +++------------------------
 3 files changed, 78 insertions(+), 129 deletions(-)

-- 
2.41.0

Re: [PATCH 0/5] Refactor few old-style XML node lookup loops

Posted by Michal Prívozník 8 months, 1 week ago

On 8/28/23 15:57, Peter Krempa wrote:
> Peter Krempa (5):
>   virNetworkDNSHostDefParseXML: Refactor parsing
>   virsh: domain: Refactor XML handling for disk changes
>   virDomainFeaturesCapabilitiesDefParse: Use virXMLNodeGetSubelementList
>   virDomainFeaturesKVMDefParse: Use virXMLNodeGetSubelementList
>   virDomainFeaturesXENDefParse: Use virXMLNodeGetSubelementList
> 
>  src/conf/domain_conf.c  | 50 +++++++++++-----------
>  src/conf/network_conf.c | 94 ++++++++++++++++++++---------------------
>  tools/virsh-domain.c    | 63 +++------------------------
>  3 files changed, 78 insertions(+), 129 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal