The "return;" statements at the end of functions do not make much sense, so let's remove them.
Cc: John Snow <js...@redhat.com> Cc: qemu-block@nongnu.org Signed-off-by: Thomas Huth <th...@redhat.com> --- 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