On Fri, 9 Mar 2018, Sergei Shtylyov wrote:
> On 03/09/2018 05:10 PM, Ilpo Järvinen wrote:
>
> > @@ -3068,8 +3072,11 @@ static int tcp_clean_rtx_queue(struct sock *sk, u32
> > prior_fack,
> > - if (!after(scb->end_seq, tp->high_seq))
> > + if (!after(scb->end_seq, tp->high_seq)) {
> > flag |= FLAG_ORIG_SACK_ACKED;
> > + } else {
> > + newdata_acked += acked_pcount;
> > + }
>
> Why add {} where they're not needed?
Not for a particular reason (I don't know even myself why I added them
here as usually I don't add such extra braces).
I'll remove them, thanks.
--
i.
