qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-devel] Re: [PATCH 14/15] Use gcc warning flag -Wempty-body, fix wa


From: Michael S. Tsirkin
Subject: [Qemu-devel] Re: [PATCH 14/15] Use gcc warning flag -Wempty-body, fix warnings
Date: Sun, 5 Sep 2010 20:54:56 +0300
User-agent: Mutt/1.5.20 (2009-12-10)

On Sun, Sep 05, 2010 at 03:07:23PM +0000, Blue Swirl wrote:
> diff --git a/hw/pxa2xx.c b/hw/pxa2xx.c
> index 88f61c0..6e04645 100644
> --- a/hw/pxa2xx.c
> +++ b/hw/pxa2xx.c
> @@ -1877,8 +1877,9 @@ static void pxa2xx_fir_write(void *opaque,
> target_phys_addr_t addr,
>          s->control[0] = value;
>          if (!(value & (1 << 4)))                     /* RXE */
>              s->rx_len = s->rx_start = 0;
> -        if (!(value & (1 << 3)))                     /* TXE */
> -            /* Nop */;
> +        if (!(value & (1 << 3))) {                      /* TXE */
> +            /* Nop */
> +        }

Should we change the surrounding code to use {} to keep it consistent?
It's annoying to have {} in one place only.

>          s->enable = value & 1;                               /* ITR */
>          if (!s->enable)
>              s->status[0] = 0;



reply via email to

[Prev in Thread] Current Thread [Next in Thread]