Merge pull request #1174 from slaclab/cpplinter

General C/C++ Updates

.github/workflows/surf_ci.yml

@@ -49,6 +49,10 @@ jobs:
           python -m compileall -f python/ scripts/ tests/
           flake8 --count python/ scripts/ tests/
 
+      - name: C/C++ Linter
+        run: |
+          find . -name '*.h' -o -name '*.cpp' -o -name '*.c' | xargs cpplint
+
       - name: VHDL Regression Testing
         run: |
           ./ghdl-build.sh > /dev/null 2>&1

.gitignore

@@ -3,6 +3,7 @@
 
 # Compiled source files
 *.o
+*.so
 *.pyc
 *.db
 *.swp

CPPLINT.cfg

@@ -0,0 +1,22 @@
+#######################################################
+# CPPLINT.cfg
+#######################################################
+
+# Increase the max number of characters on a given line
+linelength=250
+
+# List of filters to apply
+filter=-legal/copyright
+
+# Disable the build/header_guard check
+# Note changing header guard has wrong style
+# E.g. from __SSI_PRINTF__ to INCLUDE_ROGUE_UTILITIES_MODULE_H_
+filter=-build/header_guard
+
+# Disable the readability/casting check
+# Because we are using C code
+filter=-readability/casting
+
+# Disable the build/include_subdir check
+# Because headers are organized in same directory and C code
+filter=-build/include_subdir