On Fri, Feb 16, 2018 at 17:28:01 +0100, Andrea Bolognani wrote:
Some variables are only used for PCI controllers, and we're
going
to add more soon. We can declare them in the 'case' scope rather
than in the function scope to make it a bit nicer.
Signed-off-by: Andrea Bolognani <abologna(a)redhat.com>
---
src/qemu/qemu_command.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
index 5e4dfcf75..2291bf5da 100644
--- a/src/qemu/qemu_command.c
+++ b/src/qemu/qemu_command.c
@@ -2625,8 +2625,6 @@ qemuBuildControllerDevStr(const virDomainDef *domainDef,
{
virBuffer buf = VIR_BUFFER_INITIALIZER;
int address_type = def->info.type;
- const virDomainPCIControllerOpts *pciopts;
- const char *modelName = NULL;
*devstr = NULL;
Since the next patch changes both lines again I suggest you squash it
into the next patch.