[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/6] hw/net/rtl8139: Simplify if/else statement
From: |
Cédric Le Goater |
Subject: |
Re: [PATCH 3/6] hw/net/rtl8139: Simplify if/else statement |
Date: |
Fri, 6 Mar 2020 16:26:31 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 3/5/20 6:56 PM, Philippe Mathieu-Daudé wrote:
> Rewrite:
>
> if (E) {
> return A;
> } else {
> return B;
> }
> /* EOF */
> }
>
> as:
>
> if (E) {
> return A;
> }
> return B;
> }
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
Reviewed-by: Cédric Le Goater <address@hidden>
> ---
> hw/net/rtl8139.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/net/rtl8139.c b/hw/net/rtl8139.c
> index ae4739bc09..ef3211537f 100644
> --- a/hw/net/rtl8139.c
> +++ b/hw/net/rtl8139.c
> @@ -808,11 +808,11 @@ static int rtl8139_can_receive(NetClientState *nc)
> /* ??? Flow control not implemented in c+ mode.
> This is a hack to work around slirp deficiencies anyway. */
> return 1;
> - } else {
> - avail = MOD2(s->RxBufferSize + s->RxBufPtr - s->RxBufAddr,
> - s->RxBufferSize);
> - return (avail == 0 || avail >= 1514 || (s->IntrMask & RxOverflow));
> }
> +
> + avail = MOD2(s->RxBufferSize + s->RxBufPtr - s->RxBufAddr,
> + s->RxBufferSize);
> + return avail == 0 || avail >= 1514 || (s->IntrMask & RxOverflow);
> }
>
> static ssize_t rtl8139_do_receive(NetClientState *nc, const uint8_t *buf,
> size_t size_, int do_interrupt)
>
- [PATCH 0/6] hw/net: Make Net/CanBus can_receive() handlers return a boolean, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 1/6] hw/net/e1000e_core: Let e1000e_can_receive() return a boolean, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 2/6] hw/net/smc91c111: Let smc91c111_can_receive() return a boolean, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 3/6] hw/net/rtl8139: Simplify if/else statement, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 4/6] hw/net/rtl8139: Update coding style to make checkpatch.pl happy, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 5/6] hw/net: Make NetCanReceive() return a boolean, Philippe Mathieu-Daudé, 2020/03/05
- [PATCH 6/6] hw/net/can: Make CanBusClientInfo::can_receive() return a boolean, Philippe Mathieu-Daudé, 2020/03/05