[v1] Refactor more XML parsing boilerplate code, part V

[libvirt PATCH 00/10] Refactor more XML parsing boilerplate code, part V

Tim Wiederhake posted 10 patches 3 years ago

Diff against v2
Download series mbox

Test syntax-check failed

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

There is a newer version of this series

src/conf/domain_conf.c         | 643 +++++++++++----------------------
src/conf/domain_conf.h         |   8 +-
src/qemu/qemu_domain_address.c |   2 +-
src/util/virxml.c              |   2 +-
4 files changed, 227 insertions(+), 428 deletions(-)

Expand all Fold all

[libvirt PATCH 00/10] Refactor more XML parsing boilerplate code, part V

Posted by Tim Wiederhake 3 years ago

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

Tim Wiederhake (10):
  virXMLPropEnum: Fix return value
  virDomainControllerDef: Change type of ioeventfd to virTristateSwitch
  virDomainPCIControllerOpts: Change type of modelName to
    virDomainControllerPCIModelName
  virDomainControllerDefParseXML: Use virXMLProp*
  virDomainActualNetDef: Change type of type to virDomainNetType
  virDomainActualNetDefParseXML: Use virXMLProp*
  virDomainNetDefParseXML: Use virXMLProp*
  virDomainGraphicsListenDefParseXML: Use virXMLProp*
  virDomainGraphicsDef: Change type of sharePolicy to
    virDomainGraphicsVNCSharePolicy
  virDomainGraphicsDefParseXMLVNC: Use virXMLProp*

 src/conf/domain_conf.c         | 643 +++++++++++----------------------
 src/conf/domain_conf.h         |   8 +-
 src/qemu/qemu_domain_address.c |   2 +-
 src/util/virxml.c              |   2 +-
 4 files changed, 227 insertions(+), 428 deletions(-)

-- 
2.26.3

Re: [libvirt PATCH 00/10] Refactor more XML parsing boilerplate code, part V

Posted by Ján Tomko 3 years ago

On a Friday in 2021, Tim Wiederhake wrote:
>For background, see
>https://listman.redhat.com/archives/libvir-list/2021-April/msg00668.html
>
>Tim Wiederhake (10):
>  virXMLPropEnum: Fix return value
>  virDomainControllerDef: Change type of ioeventfd to virTristateSwitch
>  virDomainPCIControllerOpts: Change type of modelName to
>    virDomainControllerPCIModelName
>  virDomainControllerDefParseXML: Use virXMLProp*
>  virDomainActualNetDef: Change type of type to virDomainNetType
>  virDomainActualNetDefParseXML: Use virXMLProp*
>  virDomainNetDefParseXML: Use virXMLProp*
>  virDomainGraphicsListenDefParseXML: Use virXMLProp*
>  virDomainGraphicsDef: Change type of sharePolicy to
>    virDomainGraphicsVNCSharePolicy
>  virDomainGraphicsDefParseXMLVNC: Use virXMLProp*
>
> src/conf/domain_conf.c         | 643 +++++++++++----------------------
> src/conf/domain_conf.h         |   8 +-
> src/qemu/qemu_domain_address.c |   2 +-
> src/util/virxml.c              |   2 +-
> 4 files changed, 227 insertions(+), 428 deletions(-)
>

I have pushed all but 4/10 and 6/10.

Jano