Our coding style requires that a body of an if() longer than two lines is wrapped in a curly braces. There's one offender in qemuDomainAttachHostPCIDevice(). Fortunately, there was no functional problem because one of the lines is a comment. Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx> --- Pushed under trivial rule. src/qemu/qemu_hotplug.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c index 22acbd0852..f36de2385a 100644 --- a/src/qemu/qemu_hotplug.c +++ b/src/qemu/qemu_hotplug.c @@ -1663,9 +1663,10 @@ qemuDomainAttachHostPCIDevice(virQEMUDriver *driver, qemuAssignDeviceHostdevAlias(vm->def, &info->alias, -1); - if (qemuDomainIsPSeries(vm->def)) + if (qemuDomainIsPSeries(vm->def)) { /* Isolation groups are only relevant for pSeries guests */ qemuDomainFillDeviceIsolationGroup(vm->def, &dev); + } if (qemuDomainEnsurePCIAddress(vm, &dev) < 0) goto error; -- 2.34.1