qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 2/2 V4] spaces around '<<' everywhere


From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH 2/2 V4] spaces around '<<' everywhere
Date: Fri, 07 Mar 2014 06:33:43 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0

On 03/07/2014 06:20 AM, Romain Dolbeau wrote:
> Signed-off-by: Romain Dolbeau <address@hidden>
> ---
>  hw/net/e1000.c |   18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)

>      if (++s->eecd_state.bitnum_in == 9 && !s->eecd_state.reading) {
> -        s->eecd_state.bitnum_out = ((s->eecd_state.val_in & 0x3f)<<4)-1;
> +        s->eecd_state.bitnum_out = ((s->eecd_state.val_in & 0x3f) << 4)-1;

If you're fixing operator spacing, why not also fix the spacing around
binary '-' at the same time?


>          for (i = 0; i < 3; i++) {
> -            d->eeprom_data[i] = (macaddr[2*i+1]<<8) | macaddr[2*i];
> +            d->eeprom_data[i] = (macaddr[2*i+1] << 8) | macaddr[2*i];

Likewise around * and +

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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