[Rprotobuf-commits] r622 - /

[noreply at r-forge.r-project.org]

Author: murray
Date: 2013-12-28 00:56:26 +0100 (Sat, 28 Dec 2013)
New Revision: 622

Added:
   STYLE
Log:
Add notes about clang-format.



Added: STYLE
===================================================================
--- STYLE	                        (rev 0)
+++ STYLE	2013-12-27 23:56:26 UTC (rev 622)
@@ -0,0 +1,22 @@
+The RProtoBuf code base is a mix of many coding styles regarding
+indentation, spaces, braces, and more.
+
+Murray proposes running the whole code base through
+
+ clang-format -style="{BasedOnStyle: google, IndentWidth: 4}" -i *.cpp
+
+There are a number of other parameters that can be set however, see the full output of
+
+ clang-format -dump-config
+
+One changes have been made locally, you can view the diff with whitespace omitted with :
+
+ svn diff -x -w
+
+This will still pull up all the changes where lines were broken at 80
+characters or braces changed lines, but will ignore most of the pure
+whitespace changes for reviewing purposes before submitting.
+
+After the code base has been converted with clang-format, the emacs
+metadata comments at the top of each file will need to be updated as
+well.
\ No newline at end of file