[PATCH v2 0/6] virsh: refactor some bigger functions

Michal Prívozník mprivozn at redhat.com
Fri Sep 24 07:53:34 UTC 2021


On 9/24/21 1:25 AM, Kristina Hanicova wrote:
> This is v2 of patch:
> https://listman.redhat.com/archives/libvir-list/2021-September/msg00730.html
> 
> Diff to v1:
> * split the previous patch into more smaller ones to make the code
>   review easier
> * small code quality alternations I did not notice were possible before
> 
> This series refactors a few bigger functions mainly trying to remove too
> deep indentation and make them more readable and more consistent with
> the rest of the files.
> 
> Kristina Hanicova (6):
>   virsh: domain: refactor cmdSchedinfo()
>   virsh: domain: refactor virshCPUCountCollect()
>   virsh: domain: refactor cmdLxcEnterNamespace()
>   virsh: host: refactor cmdFreecell()
>   virsh: host: refactor cmdNodeCpuStats()
>   virsh: volume: refactor cmdVolInfo()
> 
>  tools/virsh-domain.c | 210 ++++++++++++++++++++-----------------------
>  tools/virsh-host.c   | 139 ++++++++++++++--------------
>  tools/virsh-volume.c |  52 +++++------
>  3 files changed, 189 insertions(+), 212 deletions(-)
> 

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

I've done the change suggested in 1/6 and pushed.

Michal




More information about the libvir-list mailing list