Merge pull request #15 from SSARCandy/linter

Linter

Author: hhschu

.github/workflows/c-cpp.yml

@@ -15,8 +15,10 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - name: apt
-      run: sudo apt-get install g++ cmake lcov libboost-all-dev
+      run: sudo apt-get install g++ cmake lcov libboost-all-dev clang-format
     - name: submodule
       run: git submodule update --init --recursive
+    - name: lint
+      run: bash linter.sh
     - name: test
       run: ./build.sh -t

.travis.yml

@@ -9,6 +9,7 @@ addons:
     packages: clang-format colordiff lcov libboost-all-dev
 
 script:
+  - bash linter.sh
   - ./build.sh -t
   - ./build.sh -c
 

linter.sh

@@ -0,0 +1,41 @@
+#!/bin/bash
+COLOR_REST='\e[0m';
+COLOR_BLUE='\e[1;34m';
+
+DIFF_EXE=""
+CLANG_FORMATTER=""
+AUTO_FIX="false"
+
+if [ "${1}" == '-i' ]; then
+	AUTO_FIX="true"
+fi
+
+if command -v colordiff > /dev/null; then
+	DIFF_EXE=$(command -v colordiff)
+elif command -v diff > /dev/null; then
+	DIFF_EXE=$(command -v diff)
+fi
+
+if command -v clang-format > /dev/null; then
+	CLANG_FORMATTER=$(command -v clang-format)
+else
+	echo -e "Missing clang-format, please install it with: \n    ${COLOR_BLUE}sudo apt-get install clang-format${COLOR_REST}"
+	exit 1
+fi
+
+file_list=$(find ini test \( -name '*.h' -or -name '*.cpp' \))
+return_code=0
+for f in ${file_list}; do
+    checker_cmd="${DIFF_EXE} -u <(cat ${f}) <(${CLANG_FORMATTER} ${f} -style=file)"
+	if ! diff=$(bash -c "${checker_cmd}"); then
+		echo -e ">>>>> In file ${COLOR_BLUE}${f}${COLOR_REST} <<<<<"
+		echo "${diff}" | tail -n +4
+		return_code=1
+		if [ "${AUTO_FIX}" == "true" ]; then
+			bash -c "${CLANG_FORMATTER} ${f} -i -style=file"
+			echo -e ">>>>> Auto-format ${COLOR_BLUE}${f}${COLOR_REST} done. <<<<<"
+		fi
+	fi
+done
+
+exit ${return_code}