[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