summaryrefslogtreecommitdiff
path: root/README.Coding
diff options
context:
space:
mode:
authorUri Simchoni <uri@samba.org>2016-10-02 20:40:06 +0300
committerJeremy Allison <jra@samba.org>2016-10-04 02:00:24 +0200
commit533b1210ea8e86bf90d55c01560b82f1741ca78d (patch)
tree06bd271bff5d20a92bd43b8ca6925f9e5b55886a /README.Coding
parent1b5dd78c7b43ce6d6c409cbf8b0402ee254eb950 (diff)
downloadsamba-533b1210ea8e86bf90d55c01560b82f1741ca78d.tar.gz
README.Coding: Add rule about function declaration indentation
Signed-off-by: Uri Simchoni <uri@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'README.Coding')
-rw-r--r--README.Coding6
1 files changed, 6 insertions, 0 deletions
diff --git a/README.Coding b/README.Coding
index 74dbc6eea66..2bd7552f9fe 100644
--- a/README.Coding
+++ b/README.Coding
@@ -191,6 +191,12 @@ 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
+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.
+
If, switch, & Code blocks
-------------------------