On 9/4/21 4:43 PM, Ján Tomko wrote:
Ján Tomko (24): tests: virnetdaemontest: split variable declarations conf: add cleanup func for virNetworkXMLOption conf: add cleanup func for virMacMap tests: use g_autoptr instead of virObjectUnref tests: qemu: use g_autofree tests: network: use g_autofree tests: libxl: use g_autofree tests: nwfilter: use g_autofree tests: vmx: use g_autofree tests: vir: use g_autofree tests: use g_autofree tests: use g_autoptr instead of virJSONValueFree tests: use g_auto for virCPUDataFree tests: use g_auto instead of VIR_FORCE_FCLOSE tests: use VIR_AUTOCLOSE tests: use g_auto instead of vshTableFree tests: use g_auto instead of virDomainFree tests: qemu: remove pointless labels tests: vmx: remove pointless labels tests: libxl: remove pointless labels tests: nwfilter: remove pointless labels tests: vshtabletest: remove pointless labels tests: vir: remove pointless labels
You missed removal of one cleanup: label in 23/24 (I had assumed it would give a compile warning in that case, but the CI pipeline passed it, so...)
tests: remove pointless labels
71 files changed, 628 insertions(+), 1226 deletions(-)
Beer-worthy statistics! I had a couple random comments beyond the one up above, but that's the only one that needs any action. Once you've fixed that, series: Reviewed-by: Laine Stump <laine@redhat.com>