On Wed, Sep 07, 2016 at 10:40:24AM +0800, fgao@xxxxxxxxxx wrote: > From: Gao Feng <fgao@xxxxxxxxxx> > > The origin codes perform two condition checks with dst_mtu(skb_dst(skb)) > and in_mtu. And the last statement is "min(dst_mtu(skb_dst(skb)), > in_mtu) - minlen". It may let reader think about how about the result. > Would it be negative. > > Now assign the result of min(dst_mtu(skb_dst(skb)), in_mtu) to a new > variable, then only perform one condition check, and it is more readable. Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
