On Thu, Aug 12, 2021 at 10:32:38AM +0200, Ján Tomko wrote:
>Incomprehensive series removing many cleanup sections.
>
>Patch 1/12 is common with my other series:
>[libvirt PATCH 0/3] xml: use g_auto for xmlXPathObject
>
>Ján Tomko (12):
> util: define cleanup func for xmlXPathObject
> tools: virsh: split variable declarations
> tools: virsh: cmdDominfo: rename 'ostype' variable
> tools: virsh: use automatic cleanup for virDomainObj
> tools: virsh: use automatic cleanup for xmlXPathContext
> tools: virsh: use automatic cleanup for xmlXPathObject
> tools: virsh: use automatic cleanup for xmlDoc
> tools: virsh: use automatic cleanup for vshTable
> tools: virsh: reduce variable scope to use automatic cleanup
> tools: virsh: use automatic cleanup for char **
> tools: virsh: use g_autofree
> tools: virsh: remove redundant labels
>
with the 13/12 fixup for 12/12
Reviewed-by: Martin Kletzander <mkletzan@redhat.com>
The empty lines (see PATCH 12/12) are something I feel would make it
consistent with the rest of the code, but feel free to disagree.