On 7/13/20 2:23 PM, Juan Quintela wrote: > From: Liao Pingfang <[email protected]> > > Remove superfluous breaks, as there is a "return" before them. > > Signed-off-by: Liao Pingfang <[email protected]> > Signed-off-by: Yi Wang <[email protected]> > Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
I hadn't reviewed this patch, but now I did ¯\_(ツ)_/¯ > Reviewed-by: Juan Quintela <[email protected]> > Reviewed-by: Thomas Huth <[email protected]> > Signed-off-by: Juan Quintela <[email protected]> > --- > migration/migration.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/migration/migration.c b/migration/migration.c > index 08519de56f..2ed9923227 100644 > --- a/migration/migration.c > +++ b/migration/migration.c > @@ -986,7 +986,6 @@ static void fill_source_migration_info(MigrationInfo > *info) > /* no migration has happened ever */ > /* do not overwrite destination migration status */ > return; > - break; > case MIGRATION_STATUS_SETUP: > info->has_status = true; > info->has_total_time = false; > @@ -1105,7 +1104,6 @@ static void > fill_destination_migration_info(MigrationInfo *info) > switch (mis->state) { > case MIGRATION_STATUS_NONE: > return; > - break; > case MIGRATION_STATUS_SETUP: > case MIGRATION_STATUS_CANCELLING: > case MIGRATION_STATUS_CANCELLED: >
