[v2] Refactor more XML parsing boilerplate code, part VI

[libvirt PATCH v2 0/6] Refactor more XML parsing boilerplate code, part VI

Tim Wiederhake posted 6 patches 2 years, 12 months ago

Diff against v1 v3
Download series mbox

Test syntax-check failed

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

There is a newer version of this series

src/conf/domain_conf.c                        | 609 +++++-------------
.../qemuxml2argvdata/aarch64-gic-invalid.err  |   2 +-
2 files changed, 170 insertions(+), 441 deletions(-)

Expand all Fold all

[libvirt PATCH v2 0/6] Refactor more XML parsing boilerplate code, part VI

Posted by Tim Wiederhake 2 years, 12 months ago

For background, see
https://listman.redhat.com/archives/libvir-list/2021-April/msg00668.html

V1: https://listman.redhat.com/archives/libvir-list/2021-April/msg01184.html

Changes since V1:
* Split up "virDomainFeaturesDefParse"
* Added missing VIR_XML_PROP_NONZERO
* Some line break changes
* Mentioned printf %s NULL fix in commit message
* Improved justification for parser strictening

Tim Wiederhake (6):
  virDomainGraphicsDefParseXMLSpice: Use virXMLProp*
  virDomainSoundDefParseXML: Use virXMLProp*
  virDomainFeaturesDefParse: Use virXMLPropUInt
  virDomainFeaturesDefParse: Use virXMLPropTristateSwitch
  virDomainFeaturesDefParse: Use virXMLPropEnum
  virDomainAudioSDLParse: Use virXMLProp*

 src/conf/domain_conf.c                        | 609 +++++-------------
 .../qemuxml2argvdata/aarch64-gic-invalid.err  |   2 +-
 2 files changed, 170 insertions(+), 441 deletions(-)

-- 
2.26.3