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@xxxxxxxxxx> I've done the change suggested in 1/6 and pushed. Michal