More XML parsing boiler code refactoring

[libvirt PATCH 0/6] More XML parsing boiler code refactoring

Tim Wiederhake posted 6 patches 2 years, 9 months ago

Download series mbox

Test syntax-check failed

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

src/conf/domain_conf.c | 41 +++++++++++------------------------------
src/conf/domain_conf.h |  4 +++-
2 files changed, 14 insertions(+), 31 deletions(-)

Expand all Fold all

[libvirt PATCH 0/6] More XML parsing boiler code refactoring

Posted by Tim Wiederhake 2 years, 9 months ago

In the spirit of
https://listman.redhat.com/archives/libvir-list/2021-May/msg00550.html.

Tim Wiederhake (6):
  conf: Add AUTOPTR_CLEANUP_FUNC for virDomainSEVDef
  conf: virDomainSEVDef: Change type of "sectype" to
    virDomainLaunchSecurity
  virDomainSEVDefParseXML: Use virXMLPropEnum
  virDomainSEVDefParseXML: Use automatic memory management
  virDomainSEVDefParseXML: Remove superfluous `goto`s
  virDomainSEVDefParseXML: Remove superfluous variable initialization

 src/conf/domain_conf.c | 41 +++++++++++------------------------------
 src/conf/domain_conf.h |  4 +++-
 2 files changed, 14 insertions(+), 31 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH 0/6] More XML parsing boiler code refactoring

Posted by Jano Tomko 2 years, 9 months ago

On a %A in %Y, Tim Wiederhake wrote:
> In the spirit of
> https://listman.redhat.com/archives/libvir-list/2021-May/msg00550.html.
> 
> Tim Wiederhake (6):
>   conf: Add AUTOPTR_CLEANUP_FUNC for virDomainSEVDef
>   conf: virDomainSEVDef: Change type of "sectype" to
>     virDomainLaunchSecurity
>   virDomainSEVDefParseXML: Use virXMLPropEnum
>   virDomainSEVDefParseXML: Use automatic memory management
>   virDomainSEVDefParseXML: Remove superfluous `goto`s
>   virDomainSEVDefParseXML: Remove superfluous variable initialization
> 
>  src/conf/domain_conf.c | 41 +++++++++++------------------------------
>  src/conf/domain_conf.h |  4 +++-
>  2 files changed, 14 insertions(+), 31 deletions(-)
> 

Reviewed-by: Ján Tomko <jtomko@redhat.com>
and pushed

Jano