[libvirt PATCH 00/20] use g_auto for virCommand (Episode II: The Sequel)

Ján Tomko posted 20 patches 2 years, 3 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1639420923.git.jtomko@redhat.com
src/openvz/openvz_conf.c       |  62 ++++-------
src/openvz/openvz_driver.c     | 197 +++++++++++++--------------------
src/openvz/openvz_util.c       |  23 ++--
src/vmware/vmware_conf.c       |  33 ++----
src/vmware/vmware_driver.c     |  19 +---
src/vz/vz_utils.c              |   3 +-
tests/networkxml2conftest.c    |   5 +-
tests/storagevolxml2argvtest.c |   4 +-
8 files changed, 125 insertions(+), 221 deletions(-)
[libvirt PATCH 00/20] use g_auto for virCommand (Episode II: The Sequel)
Posted by Ján Tomko 2 years, 3 months ago
TBD: docs/ and src/util

Ján Tomko (20):
  openvz: refactor openvzExtractVersionInfo
  openvz: refactor openvzLoadDomains
  openvz: refactor openvzGetVEID
  openvz: refactor openvzDomainDefineCmd
  openvz: refactor openvzSetInitialConfig
  openvz: refactor openvzSetDiskQuota
  openvz: refactor openvzDomainSetNetwork
  openvz: refactor openvzConnectListDomains
  openvz: refactor openvzConnectListDefinedDomains
  openvz: refactor openvzDomainGetBarrierLimit
  openvz: refactor openvzDomainSetBarrierLimit
  openvz: refactor openvzGetVEStatus
  openvz: use g_auto in openvzDomainMigratePerform3Params
  openvz: refactor openvzVEGetStringParam
  vz: utils: use g_auto for virCommand
  vmware: refactor vmwareUpdateVMStatus
  vmware: use g_auto in vmwareLoadDomains
  vmware: refactor vmwareExtractVersion
  tests: use g_auto in testCompareXMLToConfFiles
  tests: storagevolxml2argvtest: do not reuse cmd

 src/openvz/openvz_conf.c       |  62 ++++-------
 src/openvz/openvz_driver.c     | 197 +++++++++++++--------------------
 src/openvz/openvz_util.c       |  23 ++--
 src/vmware/vmware_conf.c       |  33 ++----
 src/vmware/vmware_driver.c     |  19 +---
 src/vz/vz_utils.c              |   3 +-
 tests/networkxml2conftest.c    |   5 +-
 tests/storagevolxml2argvtest.c |   4 +-
 8 files changed, 125 insertions(+), 221 deletions(-)

-- 
2.31.1

Re: [libvirt PATCH 00/20] use g_auto for virCommand (Episode II: The Sequel)
Posted by Michal Prívozník 2 years, 3 months ago
On 12/13/21 19:42, Ján Tomko wrote:
> TBD: docs/ and src/util
> 
> Ján Tomko (20):
>   openvz: refactor openvzExtractVersionInfo
>   openvz: refactor openvzLoadDomains
>   openvz: refactor openvzGetVEID
>   openvz: refactor openvzDomainDefineCmd
>   openvz: refactor openvzSetInitialConfig
>   openvz: refactor openvzSetDiskQuota
>   openvz: refactor openvzDomainSetNetwork
>   openvz: refactor openvzConnectListDomains
>   openvz: refactor openvzConnectListDefinedDomains
>   openvz: refactor openvzDomainGetBarrierLimit
>   openvz: refactor openvzDomainSetBarrierLimit
>   openvz: refactor openvzGetVEStatus
>   openvz: use g_auto in openvzDomainMigratePerform3Params
>   openvz: refactor openvzVEGetStringParam
>   vz: utils: use g_auto for virCommand
>   vmware: refactor vmwareUpdateVMStatus
>   vmware: use g_auto in vmwareLoadDomains
>   vmware: refactor vmwareExtractVersion
>   tests: use g_auto in testCompareXMLToConfFiles
>   tests: storagevolxml2argvtest: do not reuse cmd
> 
>  src/openvz/openvz_conf.c       |  62 ++++-------
>  src/openvz/openvz_driver.c     | 197 +++++++++++++--------------------
>  src/openvz/openvz_util.c       |  23 ++--
>  src/vmware/vmware_conf.c       |  33 ++----
>  src/vmware/vmware_driver.c     |  19 +---
>  src/vz/vz_utils.c              |   3 +-
>  tests/networkxml2conftest.c    |   5 +-
>  tests/storagevolxml2argvtest.c |   4 +-
>  8 files changed, 125 insertions(+), 221 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal