Coding Style

.githooks/pre-commit

@@ -0,0 +1,26 @@
+#!/bin/bash
+#
+# Pre-commit hook to apply clang-format to modified C/C++ source files.
+# Outputs a temporary file (/.clang-format-list) for use by a `prepare-commit-msg`
+# hook to include a list of reformatted files below main commit message
+declare -a FILES
+# Fetch files from Git and remove those flagged as deleted. Output only the file names
+CHANGED_FILES=$(git diff --cached --name-status --no-renames --word-diff=porcelain -- *.{c,h,cc,hh} | awk -F$'\t' '$1 != "D" { print $2 }')
+
+# Modify the field separator so that output of above isn't broken by filenames with spaces
+OLD_IFS=$IFS
+IFS=$'\t'
+for file in $CHANGED_FILES; do
+    # git diff add's quotes that cause issues when passing names to commands, so here we remove them
+    FILES+=(${file//\"/})
+done
+IFS=$OLD_IFS
+
+
+# Run action for each file, have to do this to prevent unusual file-names from causing issues
+IFS=$'\t\n'
+for file in $FILES; do
+    clang-format --style=file -i "${file}"
+    git add "${file}"
+    echo "${file}" >> .changed
+done

.githooks/prepare-commit-msg

@@ -0,0 +1,8 @@
+#!/bin/bash
+if [ -s .changed ]; then
+    GIT_COMMIT_FORMATTED_FILES=`cat .changed`
+    tee -a "./${1}" << EOM
+The following files were formatted by clang-format:
+${GIT_COMMIT_FORMATTED_FILES}
+EOM
+fi

debian/.gitignore

@@ -4,6 +4,7 @@
 /files
 /configure.cmdline
 /machinekit*.install
+/machinekit*.postinst
 /changelog
 
 # directory artifacts

src/.clang-format

@@ -0,0 +1,14 @@
+BreakBeforeBraces: Linux
+# BraceWrapping:
+#   AfterCaseLabel: false
+#   AfterControlStatement: true
+  # AfterEnum: false
+UseTab: Never
+TabWidth: 4
+IndentWidth: 4
+PointerAlignment: Right
+AllowAllParametersOfDeclarationOnNextLine: false
+AllowShortIfStatementsOnASingleLine: WithoutElse
+AllowShortCaseLabelsOnASingleLine: true
+AllowShortFunctionsOnASingleLine: true
+ColumnLimit: 80

src/.gitignore

@@ -24,3 +24,4 @@
 machinetalk/webtalk/zws.pb.cc
 machinetalk/webtalk/zws.pb.h
 machinetalk/tutorial/htdocs/js/machinetalk
+

src/CodingStyle

@@ -251,7 +251,7 @@ Implicit tests for zero should not be used except for boolean variables.
 Only loop control statements must be included in a for() construct.
  e.g. sum = 0;
       for (i = 0; i < 10; i++) {
-	  sum += value[i];
+	      sum += value[i];
       }
 
 NOT
@@ -265,8 +265,8 @@ boolean variables instead.
 
 The form while(true) should be used for infinite loops.
  e.g. while (true) {
-	  ...;
-      }
+	      ...;
+	  }
 
 NOT