On Mon, May 04, 2015 at 04:42:24PM +0300, Denis V. Lunev wrote:
> @@ -726,7 +727,8 @@ static void raw_refresh_limits(BlockDriverState *bs, 
> Error **errp)
>  
>      raw_probe_alignment(bs, s->fd, errp);
>      bs->bl.min_mem_alignment = s->buf_align;
> -    bs->bl.opt_mem_alignment = s->buf_align;
> +    if (bs->bl.min_mem_alignment > bs->bl.opt_mem_alignment)
> +        bs->bl.opt_mem_alignment = bs->bl.min_mem_alignment;

QEMU coding style uses {} even when the if statement body is only one
line.

Attachment: pgpLkL5myBDh3.pgp
Description: PGP signature

Reply via email to