From patchwork Mon Jan 21 12:50:55 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [5/6] hw/smc91c111: Add explicit 'return' rather than relying on fallthrough Date: Mon, 21 Jan 2013 02:50:55 -0000 From: Peter Maydell X-Patchwork-Id: 214142 Message-Id: <1358772656-7406-6-git-send-email-peter.maydell@linaro.org> To: qemu-devel@nongnu.org Cc: Blue Swirl , patches@linaro.org Add an explicit 'return' statement to a case in smc91c111_readb rather than relying on fallthrough to the following case's return statement, for code clarity and to placate static analysers. Signed-off-by: Peter Maydell --- hw/smc91c111.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/smc91c111.c b/hw/smc91c111.c index a34698f..b605069 100644 --- a/hw/smc91c111.c +++ b/hw/smc91c111.c @@ -442,6 +442,7 @@ static void smc91c111_writeb(void *opaque, hwaddr offset, return; case 12: /* Early receive. */ s->ercv = value & 0x1f; + return; case 13: /* Ignore. */ return;