From f787e79fa55affb8999ffc1765ea486c8edd46ce Mon Sep 17 00:00:00 2001 Message-Id: In-Reply-To: References: From: Blue Swirl Date: Sun, 4 Sep 2011 11:22:31 +0000 Subject: [PATCH 8/9] lsi53c895a: avoid a warning from clang analyzer Avoid this warning from clang analyzer by adjusting the scope of the variable: /src/qemu/hw/lsi53c895a.c:895:5: warning: Value stored to 'id' is never read id = (current_tag >> 8) & 0xf; Signed-off-by: Blue Swirl --- hw/lsi53c895a.c | 14 ++++++++------ 1 files changed, 8 insertions(+), 6 deletions(-) diff --git a/hw/lsi53c895a.c b/hw/lsi53c895a.c index 1643a63..9d900d0 100644 --- a/hw/lsi53c895a.c +++ b/hw/lsi53c895a.c @@ -883,7 +883,6 @@ static void lsi_do_msgout(LSIState *s) int len; uint32_t current_tag; lsi_request *current_req, *p, *p_next; - int id; if (s->current) { current_tag = s->current->tag; @@ -892,7 +891,6 @@ static void lsi_do_msgout(LSIState *s) current_tag = s->select_tag; current_req = lsi_find_by_tag(s, current_tag); } - id = (current_tag >> 8) & 0xf; DPRINTF("MSG out len=%d\n", s->dbc); while (s->dbc) { @@ -977,10 +975,14 @@ static void lsi_do_msgout(LSIState *s) device, but this is currently not implemented (and seems not to be really necessary). So let's simply clear all queued commands for the current device: */ - id = current_tag & 0x0000ff00; - QTAILQ_FOREACH_SAFE(p, &s->queue, next, p_next) { - if ((p->tag & 0x0000ff00) == id) { - scsi_req_cancel(p->req); + { + int id; + + id = current_tag & 0x0000ff00; + QTAILQ_FOREACH_SAFE(p, &s->queue, next, p_next) { + if ((p->tag & 0x0000ff00) == id) { + scsi_req_cancel(p->req); + } } } -- 1.7.2.5