From: Roopa Prabhu <ro...@cumulusnetworks.com> Date: Mon, 2 Nov 2015 13:08:49 -0800
> + if ((nh->nh_flags & RTNH_F_DEAD) || > + (nh->nh_flags & RTNH_F_LINKDOWN)) > + continue; A single "& (RTNH_F_DEAD | RTNH_F_LINKDOWN)" is more efficient and more canonical. > + if ((nh->nh_flags & RTNH_F_LINKDOWN) || > + (nh->nh_flags & RTNH_F_DEAD)) > + rt->rt_nhn_alive--; Likewise. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html