[Fwd: Re: [PATCH] s3: Fix max indentation and max column]

Uri Simchoni uri at samba.org
Wed Mar 7 18:27:50 UTC 2018

On 03/07/2018 07:16 PM, Jeremy Allison via samba-technical wrote:
> In the meantime I'll work on getting README.Coding changed
> so the "correct" way of parameter passing is preferred :-).
> Cheers,
> 	Jeremy.
How about this change (writing this in the interest of keeping the
coding style roughly clang-formattable...)?
-------------- next part --------------
diff --git a/README.Coding b/README.Coding
index e89925cad26..054cf3f1079 100644
--- a/README.Coding
+++ b/README.Coding
@@ -99,6 +99,7 @@ BreakBeforeBraces: Linux
 AllowShortIfStatementsOnASingleLine: false
 IndentCaseLabels: false
 BinPackParameters: false
+BinPackArguments: false
@@ -201,8 +202,8 @@ parameters across lines and not as encourage for gratuitous line
 splitting.  Never split a line before columns 70 - 79 unless you
 have a really good reason. Be smart about formatting.
-One exception to the previous rule is function declarations and
-definitions. In function declarations and definitions, either the
+One exception to the previous rule is function calls, declarations, and
+definitions. In function calls, declarations, and definitions, either the
 declaration is a one-liner, or each parameter is listed on its own
 line. The rationale is that if there are many parameters, each one
 should be on its own line to make tracking interface changes easier.

More information about the samba-technical mailing list