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