[libvirt] [PATCH 06/12] qemu: Move qemuNetworkIfaceConnect to qemu_interface.c and rename

[Laine Stump]

On 02/15/2016 02:37 PM, John Ferlan wrote:
> Move the misplaced function from qemu_command.c to qemu_interface.c
> since it's closer in functionality there and had less to do with building
> the command line.
>
> Rename function to qemuInterfaceNetworkConnect and modify callers.

This function really is connecting to a bridge (either one that is part 
of a libvirt network, or a host bridge defined outside libvirt), so I 
would suggest qemuInterfaceBridgeConnect(). (I had considered suggesting 
qemuInterfaceTapConnect (and the other one 
qemuInterfaceMacvlanConnect()), but I think I like naming them *Bridge* 
and *Direct* instead).

ACK with the name change.

(Of course in the end both functions should end up being called by some 
function called qemuInterfaceConnect(), and that function will be the 
only one called from qemu_command.c. No sense in going wild right out of 
the chute though, eh? :-)