makefile improvements & warnings treatment

Author: saintsHr

include/helpers.h

@@ -7,5 +7,5 @@ void wfrRefresh();
 void wfrTerminate();
 void wfrClear();
 void wfrFrameSync(unsigned int fps);
-static int wfrCompare(const void *a, const void *b);
+int wfrCompare(const void *a, const void *b);
 void wfrOrderPoints(wfrVec2 pts[4]);

makefile

@@ -1,15 +1,19 @@
 src := $(shell find src -name '*.c')
 include := -Iinclude
+debflags := -g3 -O0 -ggdb -Wall -Wextra -Wshadow -Wformat -Wconversion -DDEBUG -fsanitize=address,undefined -fno-omit-frame-pointer
+disflags := -O3 -flto -funroll-loops -finline-functions -fomit-frame-pointer -fno-strict-aliasing -fno-common -fwrapv -ffast-math -fprefetch-loop-arrays -fmerge-all-constants -ffunction-sections -fdata-sections -Wl,--gc-sections -fno-exceptions -DNDEBUG
 
 .SILENT:
 all:
 	@echo
+	@echo "Compiling in Debug Mode!"
+	@echo
 
 	@mkdir -p build lib
 
 	@for f in $(src); do \
 		echo "[Compiling]: $$f"; \
-		gcc -c $$f $(include) -o build/$$(basename $$f .c).o; \
+		gcc -c $$f $(include) $(debflags) -o build/$$(basename $$f .c).o; \
 	done
 
 	@echo
@@ -19,3 +23,24 @@ all:
 	@echo
 	@echo "Done!"
 	@echo
+
+.SILENT:
+dis:
+	@echo
+	@echo "Compiling in Distribution Mode!"
+	@echo
+
+	@mkdir -p build lib
+
+	@for f in $(src); do \
+		echo "[Compiling]: $$f"; \
+		gcc -c $$f $(include) $(disflags) -o build/$$(basename $$f .c).o; \
+	done
+
+	@echo
+	echo "[Linking]: lib/libwireframe.a"
+	ar rcs lib/libwireframe.a -lncurses build/*.o
+
+	@echo
+	@echo "Done!"
+	@echo

src/drawing/lines.c

@@ -11,7 +11,7 @@ void wfrDrawLine(wfrVec2 p1, wfrVec2 p2) {
     int dx = abs(x2 - x1);
     int dy = abs(y2 - y1);
 
-    int steep = dy > dx;
+    unsigned short steep = dy > dx;
     if (steep) {
         int tmp;
         tmp = x1; x1 = y1; y1 = tmp;
@@ -24,14 +24,14 @@ void wfrDrawLine(wfrVec2 p1, wfrVec2 p2) {
     int y = y1;
 
     wfrVec2* points = malloc(sizeof(wfrVec2));
-    int size = 0;
+    unsigned short size = 0;
 
-    points[size].x = steep ? y : x;
-    points[size].y = steep ? x : y;
+    points[size].x = (unsigned short)(steep ? y : x);
+    points[size].y = (unsigned short)(steep ? x : y);
     size++;
 
-    int x_inc = (x2 > x1) ? 1 : -1;
-    int y_inc = (y2 > y1) ? 1 : -1;
+    short x_inc = (x2 > x1) ? 1 : -1;
+    short y_inc = (y2 > y1) ? 1 : -1;
 
     for (int i = 0; i < dx; i++) {
         if (p < 0) {
@@ -44,8 +44,8 @@ void wfrDrawLine(wfrVec2 p1, wfrVec2 p2) {
 
         size++;
         points = realloc(points, size * sizeof(wfrVec2));
-        points[size - 1].x = steep ? y : x;
-        points[size - 1].y = steep ? x : y;
+        points[size - 1].x = (unsigned short)(steep ? y : x);
+        points[size - 1].y = (unsigned short)(steep ? x : y);
     }
 
     for (int i = 0; i < size; i++) {

src/helpers.c

@@ -48,7 +48,7 @@ void wfrFrameSync(unsigned int fps){
 
 static wfrVec2 wfrCenter;
 
-static int wfrCompare(const void *a, const void *b) {
+int wfrCompare(const void *a, const void *b) {
     const wfrVec2 *p1 = a;
     const wfrVec2 *p2 = b;
 
@@ -59,8 +59,11 @@ static int wfrCompare(const void *a, const void *b) {
 }
 
 void wfrOrderPoints(wfrVec2 pts[4]) {
-    wfrCenter.x = (pts[0].x + pts[1].x + pts[2].x + pts[3].x) / 4.0f;
-    wfrCenter.y = (pts[0].y + pts[1].y + pts[2].y + pts[3].y) / 4.0f;
+    float cx = ((float)pts[0].x + pts[1].x + pts[2].x + pts[3].x) / 4.0f;
+    float cy = ((float)pts[0].y + pts[1].y + pts[2].y + pts[3].y) / 4.0f;
+
+    wfrCenter.x = (unsigned short)(cx + 0.5f);
+    wfrCenter.y = (unsigned short)(cy + 0.5f);
 
     qsort(pts, 4, sizeof(wfrVec2), wfrCompare);
 }