pc: Drop useless test from isa_irq_handler
authorJan Kiszka <jan.kiszka@siemens.com>
Fri, 7 Oct 2011 07:19:34 +0000 (09:19 +0200)
committerBlue Swirl <blauwirbel@gmail.com>
Sun, 16 Oct 2011 11:10:44 +0000 (11:10 +0000)
IsaIrqState::ioapic is always non-NULL. Probably, the concrete
qemu_irq was supposed to be tested, but that's already done by
qemu_set_irq.

Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
hw/pc.c

diff --git a/hw/pc.c b/hw/pc.c
index ded475805f2af791f5b383b0d359d1627067f4e0..ff081d9342b74d22d208d49ab21d3443b70136b7 100644 (file)
--- a/hw/pc.c
+++ b/hw/pc.c
@@ -96,9 +96,8 @@ void isa_irq_handler(void *opaque, int n, int level)
     if (n < 16) {
         qemu_set_irq(isa->i8259[n], level);
     }
-    if (isa->ioapic)
-        qemu_set_irq(isa->ioapic[n], level);
-};
+    qemu_set_irq(isa->ioapic[n], level);
+}
 
 static void ioport80_write(void *opaque, uint32_t addr, uint32_t data)
 {