Introduces .clang-format (#155)

* Introduces .clang-format

Formats all files and adds a validation job to github actions.

* Fix clang format ci version

* 8 space tabs

* Don't format help content.

Author: LiquidityC

.clang-format

@@ -0,0 +1,21 @@
+---
+BasedOnStyle: LLVM
+IndentWidth: 8
+TabWidth: 8
+UseTab: ForIndentation
+BreakBeforeBraces: Linux
+ColumnLimit: 120
+PointerAlignment: Right
+IndentCaseLabels: true
+SpaceBeforeParens: ControlStatements
+AlignAfterOpenBracket: Align
+AllowShortFunctionsOnASingleLine: None
+AllowShortIfStatementsOnASingleLine: Never
+AllowShortLoopsOnASingleLine: false
+AlwaysBreakAfterReturnType: AllDefinitions
+SortIncludes: Never
+IndentPPDirectives: None
+SpaceAfterCStyleCast: false
+SpacesInParentheses: false
+SpacesInContainerLiterals: false
+BreakBeforeBinaryOperators: NonAssignment

.github/workflows/clang-format.yml

@@ -0,0 +1,23 @@
+name: clang-format
+
+on:
+  push:
+    branches: [ "dev", "feature/*" ]
+  pull_request:
+    branches: [ "dev" ]
+
+jobs:
+  clang-format:
+    name: Check formatting
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Install clang-format
+      run: pip install clang-format==21.1.8
+
+    - name: Check formatting
+      run: |
+        find src -name '*.c' -o -name '*.h' \
+          | xargs clang-format --dry-run --Werror

src/actiontext.c

@@ -21,24 +21,24 @@
 #include "util.h"
 #include "update_data.h"
 
-ActionText*
+ActionText *
 actiontext_create(Sprite *sprite)
 {
 	ActionText *t = ec_malloc(sizeof(ActionText));
-	t->pos = (Position) { 0, 0 };
+	t->pos = (Position){0, 0};
 	t->sprite = sprite;
 	t->timer = _timer_create();
 	t->dead = false;
-	t->velocity = (Vector2d) { 0, -100 };
+	t->velocity = (Vector2d){0, -100};
 	t->color = C_WHITE;
 	return t;
 }
 
 void
 actiontext_update(ActionText *t, UpdateData *data)
 {
-	t->sprite->pos.x += (int) (t->velocity.x * data->deltatime);
-	t->sprite->pos.y += (int) (t->velocity.y * data->deltatime);
+	t->sprite->pos.x += (int)(t->velocity.x * data->deltatime);
+	t->sprite->pos.y += (int)(t->velocity.y * data->deltatime);
 }
 
 void

src/actiontext.h

@@ -17,7 +17,7 @@
  */
 
 #ifndef ACTIONTEXT_H_
-#define	ACTIONTEXT_H_
+#define ACTIONTEXT_H_
 
 #include <SDL3/SDL.h>
 #include <stdbool.h>
@@ -38,12 +38,12 @@ typedef struct {
 	SDL_Color color;
 } ActionText;
 
-ActionText* actiontext_create(Sprite*);
+ActionText *actiontext_create(Sprite *);
 
-void actiontext_update(ActionText*, struct UpdateData*);
+void actiontext_update(ActionText *, struct UpdateData *);
 
-void actiontext_render(ActionText*, Camera*);
+void actiontext_render(ActionText *, Camera *);
 
-void actiontext_destroy(ActionText*);
+void actiontext_destroy(ActionText *);
 
 #endif // ACTIONTEXT_H_

src/actiontextbuilder.c

@@ -34,7 +34,7 @@ actiontextbuilder_init(SDL_Renderer *renderer)
 void
 actiontextbuilder_create_text(const char *msg, SDL_Color color, const Position *p)
 {
-	assert (gRenderer != NULL);
+	assert(gRenderer != NULL);
 	Sprite *sprite = sprite_create();
 	sprite->pos = *p;
 	sprite_load_text_texture(sprite, "GUI/SDS_8x8.ttf", 0, 11, 1);

src/actiontextbuilder.h

@@ -17,27 +17,21 @@
  */
 
 #ifndef ACTIONTEXTBUILDER_H_
-#define	ACTIONTEXTBUILDER_H_
+#define ACTIONTEXTBUILDER_H_
 
 #include <SDL3/SDL.h>
 #include "actiontext.h"
 #include "camera.h"
 #include "update_data.h"
 
-void
-actiontextbuilder_init(SDL_Renderer*);
+void actiontextbuilder_init(SDL_Renderer *);
 
-void
-actiontextbuilder_update(UpdateData*);
+void actiontextbuilder_update(UpdateData *);
 
-void
-actiontextbuilder_render(Camera*);
+void actiontextbuilder_render(Camera *);
 
-void
-actiontextbuilder_create_text(const char *msg, SDL_Color, const Position*);
+void actiontextbuilder_create_text(const char *msg, SDL_Color, const Position *);
 
-void
-actiontextbuilder_close(void);
+void actiontextbuilder_close(void);
 
 #endif // ACTIONTEXTBUILDER_H_
-

src/animation.c

@@ -25,8 +25,7 @@
 Animation *
 animation_create(unsigned int clipCount)
 {
-	Animation *animation = ec_malloc(sizeof(Animation)
-					 + clipCount * sizeof(AnimationClip));
+	Animation *animation = ec_malloc(sizeof(Animation) + clipCount * sizeof(AnimationClip));
 	animation->clipTimer = _timer_create();
 	animation->clipCount = clipCount;
 	animation->currentClip = 0;
@@ -53,9 +52,7 @@ animation_update(Animation *animation)
 		timer_start(animation->clipTimer);
 	}
 
-	if (timer_get_ticks(animation->clipTimer)
-	    >= animation->clips[animation->currentClip].renderTime)
-	{
+	if (timer_get_ticks(animation->clipTimer) >= animation->clips[animation->currentClip].renderTime) {
 		animation->currentClip++;
 		if (animation->currentClip >= animation->clipCount) {
 			animation->currentClip = 0;
@@ -67,12 +64,9 @@ animation_update(Animation *animation)
 		}
 	}
 
-	animation->sprite->clip = (SDL_Rect) {
-		animation->clips[animation->currentClip].x,
-		animation->clips[animation->currentClip].y,
-		animation->clips[animation->currentClip].w,
-		animation->clips[animation->currentClip].h
-	};
+	animation->sprite->clip =
+	    (SDL_Rect){animation->clips[animation->currentClip].x, animation->clips[animation->currentClip].y,
+	               animation->clips[animation->currentClip].w, animation->clips[animation->currentClip].h};
 }
 
 void
@@ -114,4 +108,3 @@ animation_destroy(Animation *animation)
 	sprite_destroy(animation->sprite);
 	free(animation);
 }
-

src/animation.h

@@ -17,7 +17,7 @@
  */
 
 #ifndef _ANIMATION_H
-#define	_ANIMATION_H
+#define _ANIMATION_H
 
 #include <SDL3/SDL.h>
 #include <stdbool.h>
@@ -26,17 +26,15 @@ typedef struct Timer Timer;
 typedef struct Camera Camera;
 typedef struct Sprite Sprite;
 
-typedef struct AnimationClip
-{
+typedef struct AnimationClip {
 	unsigned int x;
 	unsigned int y;
 	unsigned int w;
 	unsigned int h;
 	unsigned int renderTime;
 } AnimationClip;
 
-typedef struct Animation
-{
+typedef struct Animation {
 	Sprite *sprite;
 	Timer *clipTimer;
 	unsigned int currentClip;
@@ -46,30 +44,20 @@ typedef struct Animation
 	AnimationClip clips[];
 } Animation;
 
-Animation*
-animation_create(unsigned int clipCount);
+Animation *animation_create(unsigned int clipCount);
 
-void
-animation_load_texture(Animation *, const char *path, SDL_Renderer*);
+void animation_load_texture(Animation *, const char *path, SDL_Renderer *);
 
-void
-animation_set_frames(Animation*, const AnimationClip clips[]);
+void animation_set_frames(Animation *, const AnimationClip clips[]);
 
-void
-animation_run(Animation*);
+void animation_run(Animation *);
 
-void
-animation_update(Animation*);
+void animation_update(Animation *);
 
-void
-animation_render(Animation*, Camera*);
+void animation_render(Animation *, Camera *);
 
-void
-animation_stop(Animation*);
+void animation_stop(Animation *);
 
-void
-animation_destroy(Animation*);
+void animation_destroy(Animation *);
 
 #endif // _ANIMATION_H
-
-

src/animation_controller.c

@@ -21,22 +21,20 @@ create_explosion(Position pos)
 	sprite_set_texture(a->sprite, t, 0);
 	a->sprite->pos.x = pos.x - 16;
 	a->sprite->pos.y = pos.y - 16;
-	a->sprite->dim = (Dimension) { 64, 64 };
-	a->sprite->clip = (SDL_Rect) { 0, 0, 32, 32 };
-	a->sprite->rotationPoint = (SDL_Point) { 32, 32 };
+	a->sprite->dim = (Dimension){64, 64};
+	a->sprite->clip = (SDL_Rect){0, 0, 32, 32};
+	a->sprite->rotationPoint = (SDL_Point){32, 32};
 
 	a->loop = false;
 	a->running = true;
-	animation_set_frames(a, (AnimationClip[]) {
-			     {  0, 0, 32, 32, 100 },
-			     { 32, 0, 32, 32, 100 },
-			     { 64, 0, 32, 32, 100 },
-			     { 96, 0, 32, 32, 100 },
-			     { 128, 0, 32, 32, 100 },
-			     { 160, 0, 32, 32, 100 },
-			     { 192, 0, 32, 32, 300 },
-			     { 224, 0, 32, 32, 300 }
-			});
+	animation_set_frames(a, (AnimationClip[]){{0, 0, 32, 32, 100},
+	                                          {32, 0, 32, 32, 100},
+	                                          {64, 0, 32, 32, 100},
+	                                          {96, 0, 32, 32, 100},
+	                                          {128, 0, 32, 32, 100},
+	                                          {160, 0, 32, 32, 100},
+	                                          {192, 0, 32, 32, 300},
+	                                          {224, 0, 32, 32, 300}});
 }
 
 void

src/animation_controller.h

@@ -7,20 +7,14 @@ enum AnimationType {
 	EXPLOSION_ANIMATION,
 };
 
-void
-animation_controller_create(enum AnimationType type, Position pos);
+void animation_controller_create(enum AnimationType type, Position pos);
 
-void
-animation_controller_init(void);
+void animation_controller_init(void);
 
-void
-animation_controller_update(void);
+void animation_controller_update(void);
 
-void
-animation_controller_render(Camera *cam);
+void animation_controller_render(Camera *cam);
 
-void
-animation_controller_close(void);
-
-#endif  // ANIMATION_CONTROLLER_H_
+void animation_controller_close(void);
 
+#endif // ANIMATION_CONTROLLER_H_