[libvirt PATCHv2 00/16] refactor qemuAgentGetInterfaces
Jonathon Jongsma
jjongsma at redhat.com
Wed Oct 7 21:28:18 UTC 2020
On Wed, 7 Oct 2020 14:35:21 +0200
Ján Tomko <jtomko at redhat.com> wrote:
> v2:
> * add new patch separating error checking in
> qemuAgentGetInterfaceOneAddress
> * split out naddrs++ on a separate line
> * added documentation for qemuAgentGetInterfaceAddresses
> * added qemuAgentGetAllInterfaceAddresses
>
> Ján Tomko (16):
> qemu: agent: remove redundant checks
> qemu: agent: split out qemuAgentGetInterfaceOneAddress
> qemu: agent: remove impossible errors
> qemu: agent: reduce scope of addrs_count
> qemu: agent: expand addrs upfront
> qemu: agent: use g_auto for ifname
> qemu: agent: use virHashNew
> qemu: agent: simplify access to ifaces_ret
> qemu: agent: split out qemuAgentGetInterfaceAddresses
> qemu: agent: use GetArray to remove a check
> qemu: agent: use g_auto in qemuAgentGetInterfaces
> qemu: agent: remove cleanup in qemuAgentGetInterfaces
> qemuAgentGetInterfaceAddresses: turn ifname into char*
> qemu: agent: rename tmp_iface to iface_obj
> qemuAgentGetInterfaceOneAddress: check for errors early
> qemu: agent: split out qemuAgentGetAllInterfaceAddresses
>
> src/qemu/qemu_agent.c | 355
> +++++++++++++++++++++--------------------- 1 file changed, 180
> insertions(+), 175 deletions(-)
>
Looks good to me.
Reviewed-by: Jonathon Jongsma <jjongsma at redhat.com>
More information about the libvir-list
mailing list