On 11/10/2015 03:16 PM, Thomas Huth wrote: > 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) >
Fair enough. Reviewed-by: John Snow <js...@redhat.com>