Skip to content

Commit

Permalink
Massive style/whitespace overhaul
Browse files Browse the repository at this point in the history
* Move to four-space indentation
* Consistentify style
* Remove svn/cvs keyword strings
* Update styleguide

Tested-by: Scott Talbert <swt@techie.net>
Signed-off-by: Phil Dibowitz <phil@ipom.com>
  • Loading branch information
jaymzh committed May 10, 2013
1 parent b4ecf5d commit 96a502b
Show file tree
Hide file tree
Showing 34 changed files with 6,913 additions and 7,033 deletions.
33 changes: 30 additions & 3 deletions CodingStyle
@@ -1,7 +1,5 @@
INDENTATION
Concordance currently uses tab-character indentation. While 4- or 8-space
indentation would be preferrable, it's not what we have now, so please keep
the existing indentation.
Concordance currently uses 4-space indentation.


PAGE-WIDTH
Expand Down Expand Up @@ -29,6 +27,35 @@ and not:
The same thing for i[2] as opposed to i[ 2 ].


INDENTATION
When breaking lines that are arguments to a function or parameters to a
definition, align the second row with the columb the first row began at:

some_function(one, two, three, four, five, six, seven, eight, nine, ten,
eleven, twelve);

If your function call is nested or very long and you'd rather not do that, you
must start the first argument on it's own line like so:

some_function(
one, two, three, four, five, six, seven, eight, nine, ten, eleven, twelve,
thirteen);


BRACKETS
Do not use brackets around single-line if-statements unless it would be very
unclear without them. In otherwords, do this:

if (condition)
thing();

and not:

if (condition) {
thing();
}


CROSS-PLATFORM NOTES
Code MUST be portable and cross-platform. Our codebase works on Linux, Mac OS
X, BSD, Windows, and possibly others. As such it is important ensure any
Expand Down

0 comments on commit 96a502b

Please sign in to comment.