On 01/22/2016 02:11 PM, Cole Robinson wrote:
On 01/22/2016 02:09 PM, Cole Robinson wrote:
Nowadays we only support qemu 0.12.0+ which provides QEMU_CAPS_DEVICE, so this is all dead code. --- src/qemu/qemu_hotplug.c | 480 +++++++++++++++--------------------------------- 1 file changed, 144 insertions(+), 336 deletions(-)
git show -w attached
Just nitpicking, but seeing it this way allowed me to randomly notice that the changes have created several instances of code like this: if (!detach->info.alias) { if (qemuAssignDeviceControllerAlias(vm->def, priv->qemuCaps, detach) < 0) goto cleanup; } Maybe combine those like this: if (!detach->info.alias && (qemuAssignDeviceControllerAlias(vm->def, priv->qemuCaps, detach) < 0)) goto cleanup;