On 6/1/15, 3:58 PM, David Miller wrote:
From: Roopa Prabhu <ro...@cumulusnetworks.com>
Date: Wed, 27 May 2015 11:37:05 -0700

-       if (rtm->rtm_scope != RT_SCOPE_UNIVERSE)
-               goto errout;
+       if (nlh->nlmsg_type != RTM_DELROUTE &&
+               rtm->rtm_scope != RT_SCOPE_UNIVERSE)
+                       goto errout;
+
This is not indented correctly.

When a conditional spans multiple lines, two things must happen:

1) Each line must end with an operator rather than begin with one.

2) Each line starting with the second must start exactly at the
    first column after the openning parenthesis of the conditional.
    You must use the appropriate number of TAB then SPACE characters
    necessary to achieve this.

sorry, missed this. I have resubmitted the series. thanks.
--
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

Reply via email to