Re: [libvirt PATCH 00/12] tools: virsh: use g_auto more (glib chronicles)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>

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.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux