From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50681) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZwFL5-0006uU-6w for qemu-devel@nongnu.org; Tue, 10 Nov 2015 15:16:24 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZwFL4-0005G1-Bs for qemu-devel@nongnu.org; Tue, 10 Nov 2015 15:16:23 -0500 From: Thomas Huth Date: Tue, 10 Nov 2015 21:16:08 +0100 Message-Id: <1447186571-27672-2-git-send-email-thuth@redhat.com> In-Reply-To: <1447186571-27672-1-git-send-email-thuth@redhat.com> References: <1447186571-27672-1-git-send-email-thuth@redhat.com> Subject: [Qemu-devel] [PATCH 1/4] hw/ide: Remove superfluous return statements List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org, qemu-trivial@nongnu.org Cc: John Snow , qemu-block@nongnu.org The "return;" statements at the end of functions do not make much sense, so let's remove them. Cc: John Snow Cc: qemu-block@nongnu.org Signed-off-by: Thomas Huth --- hw/ide/atapi.c | 1 - hw/ide/macio.c | 2 -- 2 files changed, 3 deletions(-) diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c index 747f466..347c6e7 100644 --- a/hw/ide/atapi.c +++ b/hw/ide/atapi.c @@ -737,7 +737,6 @@ static void cmd_inquiry(IDEState *s, uint8_t *buf) out: buf[size_idx] = idx - preamble_len; ide_atapi_cmd_reply(s, idx, max_len); - return; } static void cmd_get_configuration(IDEState *s, uint8_t *buf) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index 893c9b9..8a56115 100644 --- a/hw/ide/macio.c +++ b/hw/ide/macio.c @@ -288,8 +288,6 @@ static void pmac_ide_atapi_transfer_cb(void *opaque, int ret) done: block_acct_done(blk_get_stats(s->blk), &s->acct); io->dma_end(opaque); - - return; } static void pmac_ide_transfer_cb(void *opaque, int ret) -- 1.8.3.1