Sergei Shtylyov <sergei.shtyl...@cogentembedded.com> wrote:

> >             case RXRPC_ACK_TYPE_NACK:
> >                     if (anno_type == RXRPC_TX_ANNO_NAK)
> >                             continue;
> > +                   if (anno_type == RXRPC_TX_ANNO_RETRANS)
> > +                           continue;
> 
>    Why not fold the above 2 *if*s together? Or use *else if* at least?

I have a pending patch that adds something between them.

David

Reply via email to