Code quality and warning removal

Solved all code quality issues, removed dead code.

Correctly freed fps buffer in texture_show_fps.c

Correctly checked for argc size before accessing argv (did not show up
in warnings but it's still dangerous)

Reordered some resource acquisition operations in examples in order to
avoid framebuffer creation if wrong inputs are specified (we fail
first).

1 files changed, 14 insertions, 6 deletions

diff --git a/examples/texture.c b/examples/texture.c
index afa0985..f3e0f0c 100644
--- a/examples/texture.c
+++ b/examples/texture.c
@@ -6,19 +6,25 @@
 
 int main(int argc, char **argv)
 {
-	// Initialize framebuffer
-	fbgl_t framebuffer;
-	if (fbgl_init(NULL, &framebuffer) != 0) {
-		fprintf(stderr, "Failed to initialize framebuffer.\n");
+	if (argc < 2) {
+		fprintf(stderr, "Usage: %s <texture_path>\n", argv[0]);
 		return EXIT_FAILURE;
 	}
 
 	// Load a TGA texture
 	const char *texture_path = argv[1];
+
 	fbgl_tga_texture_t *texture = fbgl_load_tga_texture(texture_path);
 	if (!texture) {
 		fprintf(stderr, "Failed to load texture.\n");
-		fbgl_destroy(&framebuffer);
+
+		return EXIT_FAILURE;
+	}
+
+	// Initialize framebuffer
+	fbgl_t framebuffer;
+	if (fbgl_init(NULL, &framebuffer) != 0) {
+		fprintf(stderr, "Failed to initialize framebuffer.\n");
 		return EXIT_FAILURE;
 	}
 
@@ -55,8 +61,10 @@ int main(int argc, char **argv)
 		    texture_y + texture->height >= framebuffer.height) {
 			dy = -dy; // Reverse vertical direction when hitting the top or bottom edge
 		}
+		nanosleep(
+			(struct timespec[]){ { 0, (int)5e7 } },
+			NULL); // Delay to make the marquee effect visible (adjust as needed)
 
-		usleep(50000); // Delay to make the marquee effect visible (adjust as needed)
 		framesize--;
 	}