From: Ben Pfaff <blp@nicira.com>
Date: Mon, 19 Apr 2010 21:37:56 +0000 (-0700)
Subject: CodingStyle: Drop advice about breaking lines before binary operators.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=ede818437a16d1cf50c5e8f285593f85cd6f2513;p=openvswitch

CodingStyle: Drop advice about breaking lines before binary operators.

I like the style that was prescribed here--I find it slightly easier to
read--but everyone else who submits code seems to prefer breaking
lines after binary operators instead.  No point in fighting the tide.
---

diff --git a/CodingStyle b/CodingStyle
index f4765ad8..bbb492e3 100644
--- a/CodingStyle
+++ b/CodingStyle
@@ -412,13 +412,8 @@ Exception 1: Put a space after (but not before) the "sizeof" keyword.
 Exception 2: Put a space between the () used in a cast and the
 expression whose type is cast: (void *) 0.
 
-  Break long lines before binary operators and the ternary operators ?
-and :, rather than after them, e.g.
-
-    if (first_long_condition() || second_long_condition()
-        || third_long_condition())
-
-and
+  Break long lines before the ternary operators ? and :, rather than
+after them, e.g.
 
     return (out_port != VIGP_CONTROL_PATH
             ? alpheus_output_port(dp, skb, out_port)