On 7/22/25 17:12, Kirill Shchetiniuk via Devel wrote:
This patch series is supposed to refactor the existing logic of
some parsing functions, changing the previous approach of getting
the string fisrt and then parsing the string itself. Now the
parsing logic is implemented by using the appropriate virXMLProp*
functions.
Changelog fromm v1:
1. 'ret' temp value is now 'rc'
2. code style fixes
3. 'id' param is now parsed as not mandatory
Kirill Shchetiniuk (4):
conf: virNetDevVPortProfileParse refactor
util: virSecretLookupParseSecret refactor
conf: virDomainChrDefParseTargetXML refactor
qemu: qemuDomainObjPrivateXMLParseVcpu refactor
src/conf/domain_conf.c | 16 +---
src/conf/netdev_vport_profile_conf.c | 112 +++++++++++----------------
src/qemu/qemu_domain.c | 16 +---
src/util/virsecret.c | 19 ++---
4 files changed, 55 insertions(+), 108 deletions(-)
--
2.49.0
I'm fixing all the small nits I've raised and merging.
Reviewed-by: Michal Privoznik <mprivozn(a)redhat.com>
Michal