Fixed a few annoying compiler warnings

.gitignore

@@ -72,3 +72,6 @@ fps_example
 EBOOT.PBP
 PARAM.SFO
 *.prx
+
+# dev tools
+.vscode

examples/ray_collision_2d.h

@@ -60,6 +60,8 @@ extern "C"
 
 				return true;
 			}
+
+			return false;
 		} 
 		else
 		{
@@ -198,7 +200,7 @@ extern "C"
 
 			float length = -INFINITY;
 
-			bool intersect = CheckCollisionRay2dRay2d(ray, edgeRay, &length);
+			CheckCollisionRay2dRay2d(ray, edgeRay, &length);
 
 			Vector2 nearest = Vector2Add(ray.Origin, Vector2Scale(ray.Direction, length));
 

makefile

@@ -3,7 +3,7 @@ LIBNAME = kolibri
 VERSION = 0.1.0
 
 # Compiler
-CC = gcc-14
+CC = gcc
 AR = ar
 ARFLAGS = rcs
 
@@ -45,7 +45,7 @@ OBJS_RELEASE = $(SRCS:$(SRCDIR)/%.c=$(OBJDIR)/release/%.o)
 OBJS_DEBUG   = $(SRCS:$(SRCDIR)/%.c=$(OBJDIR)/debug/%.o)
 
 # Compiler flags
-CFLAGS_COMMON = -I$(INCDIR) -I./examples -Wall -Wextra -Wpedantic
+CFLAGS_COMMON = -I$(INCDIR) -I./examples -Wall -Wextra -Wpedantic -Werror
 CFLAGS_RELEASE = $(CFLAGS_COMMON) -O3 -DNDEBUG
 CFLAGS_DEBUG = $(CFLAGS_COMMON) -g3 -O0 -DDEBUG
 

src/collision.c

@@ -371,7 +371,7 @@ CollisionScene__checkCollision(
 	);
 
 	/* Check AABB collision with each candidate */
-	for (int i = 0; i < DynamicArray_length(candidates); i++) {
+	for (size_t i = 0; i < DynamicArray_length(candidates); i++) {
 		Entity *other = candidates[i];
 		if (other == entity) continue; /* Skip self */
 		if (!other->collision_shape) continue;
@@ -813,7 +813,7 @@ CollisionScene__moveEntity(
 			bounds
 		);
 
-    for (int i = 0; i < DynamicArray_length(candidates); i++) {
+    for (size_t i = 0; i < DynamicArray_length(candidates); i++) {
         Entity *other = candidates[i];
         if (other == entity || !other->collision_shape) continue;
 
@@ -1095,7 +1095,7 @@ CollisionScene__raycast(CollisionScene *scene, K_Ray ray, Entity *ignore)
 			bbox
 		);
 
-	for (int i = 0; i < DynamicArray_length(candidates); i++) {
+	for (size_t i = 0; i < DynamicArray_length(candidates); i++) {
 		Entity *entity = candidates[i];
 
 		/* Skip ignored entity */

src/engine.c

@@ -433,7 +433,6 @@ void
 Engine__insertEntity(Engine *self, EntityNode *node)
 {
     if (MAX_NUM_ENTITIES <= self->entity_count) return;
-	Entity *entity = NODE_TO_ENTITY(node);
 	if (!self->entities) {
 		self->entities = node;
 	}

src/entity.c

@@ -356,7 +356,8 @@ EntityNode__remove(EntityNode *self)
 		*prev = self->prev,
 		*next = self->next;
 
-	DBG_OUT("EntityNode__remove: self=%p, prev=%p, next=%p", self, prev, next);
+	DBG_OUT("EntityNode__remove: self=%p, prev=%p, next=%p",
+		    (void*)self, (void*)prev, (void*)next);
 
 	next->prev = prev;
 	prev->next = next;

src/renderer.c

@@ -210,7 +210,7 @@ Renderer__queryFrustum(
     );
 
     /* Optimized culling loop */
-    for (int i = 0; i < DynamicArray_length(candidates) && *visible_count < VIS_QUERY_SIZE; i++) {
+    for (size_t i = 0; i < DynamicArray_length(candidates) && *visible_count < VIS_QUERY_SIZE; i++) {
         RenderableWrapper *wrapper = candidates[i];
 
         if (wrapper->is_entity && !wrapper->entity->visible) continue;

src/spatialhash.c

@@ -214,7 +214,7 @@ SpatialHash_queryRegion(
                     bool is_duplicate = false;
                     size_t count = DynamicArray_length(query_results);
 
-                    for (int i = 0; i < count; i++) {
+                    for (size_t i = 0; i < count; i++) {
                         if (query_results[i] == entry->data) {
                             is_duplicate = true;
                             break;