Re: [PATCH 00/11] conf: Misc cleanups & reworks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a Monday in 2022, Michal Privoznik wrote:
As I was going through domain_conf.c trying to change the code to use
virXMLPropEnum() more I had to write couple of clean ups first.

Patches can be also found here:

https://gitlab.com/MichalPrivoznik/libvirt/-/commits/virxmlprop/

but on the branch, there are also said virXMLPropEnum() patches so look
only at the first 11 patches on the branch.

Michal Prívozník (11):
 virDomainInputDefParseXML: Move validation into validator
 virDomainChrSourceDefCopy: Copy more struct members
 virDomainChrSourceDefCopy: Don't check arguments against NULL
 virDomainChrSourceDefCopy: return void
 conf: Fix type of @present in _virDomainTimerDef struct
 conf: Fix @tickpolicy member of _virDomainTimerDef struct
 conf: Fix @track member of _virDomainTimerDef struct
 conf: Fix @mode member of _virDomainTimerDef struct
 conf: Rework virDomainTimerDefFormat()
 virDomainTimerDefFormat: return void
 conf: Separate out virDomainClockDef formatting

src/conf/domain_conf.c     | 296 ++++++++++++++-----------------------
src/conf/domain_conf.h     |  22 +--
src/conf/domain_validate.c |  67 ++++++++-
src/libxl/libxl_conf.c     |   2 +-
src/libxl/xen_common.c     |  23 +--
src/lxc/lxc_cgroup.c       |   2 +-
src/lxc/lxc_controller.c   |   2 +-
src/qemu/qemu_command.c    |  28 ++--
src/qemu/qemu_process.c    |   5 +-
src/qemu/qemu_validate.c   |  14 +-
10 files changed, 226 insertions(+), 235 deletions(-)


Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux