Re: [PATCH 08/12] block/vmdk: Remove superfluous breaks

[Thomas Huth]

On 13/07/2020 11.05, Yi Wang wrote:
> From: Liao Pingfang <liao.pingfang@zte.com.cn>
> 
> Remove superfluous breaks, as there is a "return" before them.
> 
> Signed-off-by: Liao Pingfang <liao.pingfang@zte.com.cn>
> Signed-off-by: Yi Wang <wang.yi59@zte.com.cn>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  block/vmdk.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 28cec50..8f222e3 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -1053,14 +1053,11 @@ static int vmdk_open_sparse(BlockDriverState *bs, 
> BdrvChild *file, int flags,
>      switch (magic) {
>          case VMDK3_MAGIC:
>              return vmdk_open_vmfs_sparse(bs, file, flags, errp);
> -            break;
>          case VMDK4_MAGIC:
>              return vmdk_open_vmdk4(bs, file, flags, options, errp);
> -            break;
>          default:
>              error_setg(errp, "Image not in VMDK format");
>              return -EINVAL;
> -            break;
>      }
>  }
>  
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>