clang-format implementation

.github/workflows/push.yml

@@ -55,6 +55,7 @@ jobs:
           name: metadata
           path: source/Hexfile/${{ matrix.model }}.json
 
+
   build_multi-lang:
     runs-on: ubuntu-20.04
     container:
@@ -106,6 +107,7 @@ jobs:
           name: metadata
           path: source/Hexfile/${{ matrix.model }}_multi-lang.json
 
+
   tests:
     runs-on: ubuntu-20.04
     container:
@@ -133,14 +135,15 @@ jobs:
           chmod +x brieflz_test.py
           ./brieflz_test.py
 
-  check_formatting:
+
+  check_python:
     runs-on: ubuntu-20.04
     container:
       image: alpine:3.16
 
     steps:
       - name: deps
-        run: apk add --no-cache python3 py3-pip make git black sed diffutils
+        run: apk add --no-cache python3 py3-pip make git black sed diffutils findutils clang-extra-tools
 
       - uses: actions/checkout@v3
         with:
@@ -149,9 +152,6 @@ jobs:
       - name: install black
         run: python3 -m pip install bdflib flake8
 
-      - name: Check formatting with clang-format
-        run: cd source && make clean && make check-style
-
       - name: Check python formatting with black
         run: black --check Translations
 
@@ -161,8 +161,26 @@ jobs:
       - name: Check autogenerated Documentation/README.md
         run: /bin/sh ./scripts/deploy.sh docs_readme
 
-  shellcheck:
-    name: runner / shellcheck
+
+  check_c-cpp:
+    runs-on: ubuntu-20.04
+    container:
+      image: alpine:3.16
+
+    steps:
+      - name: deps
+        run: apk add --no-cache make git diffutils findutils clang-extra-tools
+
+      - uses: actions/checkout@v3
+        with:
+          submodules: true
+
+      - name: Check formatting with clang-format
+        run: cd source && make clean && make check-style
+
+
+  check_shell:
+    name: check_shell
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3

source/Makefile

@@ -794,16 +794,20 @@ style:
 	@echo "Done"
 
 check-style:
-	@for src in $(ALL_SOURCE) $(ALL_INCLUDES) ; do \
+	@ret=0; for src in $(ALL_SOURCE) $(ALL_INCLUDES) ; do \
 		var=`clang-format "$$src" | diff "$$src" - | wc -l` ; \
 		if [ $$var -ne 0 ] ; then \
 			echo "$$src does not respect the coding style (diff: $$var lines)" ; \
 			clang-format "$$src" | diff "$$src" -; \
-			exit 1 ; \
+			ret=1 ; \
 		fi ; \
-	done
-	@echo "Style check passed"
-
+	done ; \
+	if [ $$ret -eq 0 ] ; then \
+		echo "Style check passed" && exit 0 ; \
+	else \
+		echo "Style check failed at least once! Please, check the log above for the details." && exit 1 ; \
+	fi ;
+
 .PHONY: style all clean default clean-all
 .SECONDARY: