tools: add .editorconfig

.github/workflows/build.yml

@@ -0,0 +1,49 @@
+name: Clang Build CI
+
+concurrency:
+  group: build-${{ github.ref }}
+  cancel-in-progress: true
+
+on:
+  workflow_dispatch:
+  push:
+    paths:
+      - '.github/workflows/build.yml'
+      - 'src/*.c'
+      - 'src/*.h'
+      - 'configure.ac'
+      - 'Makefile.am'
+
+env:
+  GCC_ARGS: src/*.c src/*.h -Ofast
+  TARGET: margen
+  TEST_ARGS: ""
+
+jobs:
+  build-release:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v4
+      - name: aclocal
+        run: aclocal
+      - name: autoconf
+        run: autoconf
+      - name: automake
+        run: automake --add-missing
+      - name: configure
+        run: ./configure
+      - name: make
+        run: make
+      - name: make distcheck
+        run: make distcheck
+
+  run:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v5
+      - name: gcc
+        run: mkdir -p build && gcc $GCC_ARGS -o build/$TARGET
+      - name: run program
+        run: ./build/$TARGET $TEST_ARGS

.github/workflows/ci.yml

@@ -1,33 +0,0 @@
-name: C-lang CI
-
-on:
-  push:
-    branches: ["master"]
-  pull_request:
-    branches: ["master"]
-
-jobs:
-  lint:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: gcc
-        run: mkdir -p build && gcc -v -Wall -Wextra -Werror src/*.c src/*.h -lm
-
-  build-release:
-    needs: lint
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: aclocal
-        run: aclocal
-      - name: autoconf
-        run: autoconf
-      - name: automake
-        run: automake --add-missing
-      - name: configure
-        run: ./configure
-      - name: make
-        run: make
-      - name: make distcheck
-        run: make distcheck

.github/workflows/lint.yml

@@ -0,0 +1,25 @@
+name: Clang Lint CI
+
+concurrency:
+  group: lint-${{ github.ref }}
+  cancel-in-progress: true
+
+on:
+  workflow_dispatch:
+  push:
+    paths:
+      - '.github/workflows/lint.yml'
+      - 'src/*.c'
+      - 'src/*.h'
+
+env:
+  GCC_ARGS: src/*.c src/*.h -Ofast
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v5
+      - name: gcc
+        run: gcc -v -Wall -Wextra -Werror $GCC_ARGS

.gitignore

@@ -23,4 +23,5 @@ margen
 configure
 src/margen*
 *.pkg.tar.zst
-pkg
+pkg
+margen-*

Makefile.dev

@@ -9,7 +9,7 @@ clean:
 
 build:
 	@mkdir -p build
-	gcc -v -Wall -Wextra src/*.c src/*.h -lm -o build/$(TARGET)
+	gcc -Wall -Wextra -Og -g src/*.c src/*.h -o build/$(TARGET)
 
 .PHONY: install
 install: build
@@ -24,12 +24,49 @@ valgrind: build
 	valgrind --leak-check=full -s ./build/$(TARGET) $(TEST_ARGS)
 
 .PHONY: release
-release:
+release: clean
 	aclocal
 	autoconf
 	automake --add-missing
 	./configure
 	make distcheck
+	mkdir -p build
+	cp $(TARGET)-*.tar.gz build
+	@rm -rf \
+		autom4te.cache \
+		aclocal.m4 \
+		compile \
+		config.* \
+		configure \
+		depcomp \
+		$(TARGET) \
+		$(TARGET)-*.tar.gz \
+		$(TARGET)-*.pkg.tar.zst \
+		install-sh \
+		Makefile \
+		Makefile.in \
+		missing \
+		src/.* \
+		src/*.o
+
+.PHONY: release
+release-clean:
+	@rm -rf \
+		autom4te.cache \
+		aclocal.m4 \
+		compile \
+		config.* \
+		configure \
+		depcomp \
+		$(TARGET) \
+		$(TARGET)-*.tar.gz \
+		$(TARGET)-*.pkg.tar.zst \
+		install-sh \
+		Makefile \
+		Makefile.in \
+		missing \
+		src/.* \
+		src/*.o
 
 .PHONY: release-arch
 release-arch: clean

PKGBUILD

@@ -1,11 +1,11 @@
 pkgname=margen
-pkgver=1.2.5
+pkgver=1.2.6
 pkgrel=1
 pkgdesc="Generate a marble-like pattern bitmap image, blazing fast."
 arch=('i686' 'pentium4' 'x86_64' 'arm' 'armv7h' 'armv6h' 'aarch64' 'riscv64')
 url="https://github.com/klemek/margen"
 source=("${pkgname}-${pkgver}.tar.gz::https://github.com/klemek/margen/releases/download/v${pkgver}/${pkgname}-${pkgver}.tar.gz")
-sha256sums=('ff4375dacec14bce1f67bb6964232c54b184f820aaad02fda4313c58d78133c6')
+sha256sums=('e35368e963dbc81e94c4fab50cf2aa5d0f56caf6079e204ca4608eb9835670bc')
 srcdir=build
 
 build() {

README.md

@@ -1,3 +1,5 @@
+[![](https://git.klemek.fr/klemek/margen/actions/workflows/lint.yml/badge.svg?branch=main&style=flat-square)](https://git.klemek.fr/klemek/margen/actions?workflow=lint.yml) [![](https://git.klemek.fr/klemek/margen/actions/workflows/build.yml/badge.svg?branch=main&style=flat-square)](https://git.klemek.fr/klemek/margen/actions?workflow=build.yml)
+
 # margen
 
 > generate a marble-like pattern bitmap image, blazing fast.
@@ -78,14 +80,14 @@ $EDITOR configure.ac
 # make full build
 make -f Makefile.dev release
 # update PKGBUILD with new version and sha256 sum
-sha256sum margen-x.y.z.tar.gz
+sha256sum build/margen-x.y.z.tar.gz
 $EDITOR PKGBUILD
 # push to repo
 git commit -am "margen vX.Y.Z"
-git tag vX.Y.Z
+git tag vX.Y.Z -m "margen vX.Y.Z"
 git push origin master --tags
 # create release from tag on github
 # attach .tar.gz to the github release
 make -f Makefile.dev release-arch
 # attach .pkg.tar.zst to the github release
-```
+```

configure.ac

@@ -1,4 +1,4 @@
-AC_INIT([margen], [1.2.5], [klemek.dev@proton.me])
+AC_INIT([margen], [1.2.6], [klemek.dev@proton.me])
 AM_INIT_AUTOMAKE
 AC_PROG_CC
 AC_CHECK_HEADERS([stdio.h])
@@ -6,5 +6,6 @@ AC_CHECK_HEADERS([stdlib.h])
 AC_CHECK_HEADERS([stdbool.h])
 AC_CHECK_HEADERS([string.h])
 AC_CHECK_HEADERS([time.h])
+AC_CHECK_HEADERS([limits.h])
 AC_CONFIG_FILES([Makefile])
 AC_OUTPUT

src/args.c

@@ -1,13 +1,16 @@
-#include "args.h"
-#include "config.h"
-#include "rand.h"
+#include <limits.h>
 #include <stdbool.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <time.h>
 
-void print_help(int status_code) {
+#include "args.h"
+#include "config.h"
+#include "rand.h"
+#include "types.h"
+
+static void print_help(int status_code) {
   puts(PACKAGE
        " " VERSION "\n\n"
        "usage: " PACKAGE " "
@@ -46,80 +49,80 @@ void print_help(int status_code) {
   exit(status_code);
 }
 
-void invalid_arg(char *arg) {
+static void invalid_arg(char *arg) {
   fprintf(stderr, "invalid argument: '%s'\n\n", arg);
-  print_help(1);
+  print_help(EXIT_FAILURE);
 }
 
-void invalid_value(char *arg, char *value) {
+static void invalid_value(char *arg, char *value) {
   fprintf(stderr, "invalid value for argument '%s': '%s'\n\n", arg, value);
-  print_help(1);
+  print_help(EXIT_FAILURE);
 }
 
-bool is_arg(char *arg, char *ref) { return strcoll(arg, ref) == 0; }
+static bool is_arg(char *arg, char *ref) { return strcoll(arg, ref) == 0; }
 
-char *split_arg_value(char *arg) {
+static char *split_arg_value(char *arg) {
   strtok(arg, "=");
   return strtok(NULL, "=");
 }
 
-bool is_digit(char c) { return c >= '0' && c <= '9'; }
+static bool is_digit(char c) { return c >= '0' && c <= '9'; }
+
+static bool is_number(char *value) {
+  unsigned long value_len;
+  unsigned int i;
 
-bool is_number(char *value) {
   if (value == NULL) {
     return false;
   }
-  unsigned long value_len = strlen(value);
-  unsigned int i;
+  value_len = strlen(value);
   for (i = 0; i < value_len; i++) {
     if (!is_digit(value[i])) {
       return false;
     }
   }
+
   return true;
 }
 
-unsigned char parse_char(char *arg, char *value) {
+static unsigned int parse_uint(char *arg, char *value) {
+  unsigned long long tmp_value;
+
   if (!is_number(value)) {
     invalid_value(arg, value);
   }
-  unsigned long long tmp_value = (unsigned long long)atoll(value);
-  if (tmp_value >= 256) {
+  tmp_value = (unsigned long long)atoll(value);
+  if (tmp_value >= UINT_MAX) {
     invalid_value(arg, value);
   }
-  return (unsigned char)tmp_value;
+
+  return (unsigned int)tmp_value;
 }
 
-unsigned short parse_ushort(char *arg, char *value) {
-  if (!is_number(value)) {
-    invalid_value(arg, value);
-  }
-  unsigned long long tmp_value = (unsigned long long)atoll(value);
-  if (tmp_value >= 65536) {
-    invalid_value(arg, value);
-  }
-  return (unsigned short)tmp_value;
-}
-
-unsigned long parse_ulong(char *arg, char *value) {
+static unsigned long parse_ulong(char *arg, char *value) {
   if (!is_number(value)) {
     invalid_value(arg, value);
   }
   return (unsigned long)atoll(value);
 }
 
-void parse_color(char *arg, char *value, unsigned char color[3]) {
+static void parse_color(char *arg, char *value, unsigned int color[3]) {
   char *tmp;
+
   tmp = strtok(value, ",");
-  color[0] = parse_char(arg, tmp);
+  color[0] = parse_uint(arg, tmp);
   tmp = strtok(NULL, ",");
-  color[1] = parse_char(arg, tmp);
+  color[1] = parse_uint(arg, tmp);
   tmp = strtok(NULL, ",");
-  color[2] = parse_char(arg, tmp);
+  color[2] = parse_uint(arg, tmp);
 }
 
-parameters parse_args(int argc, char **argv) {
-  parameters params;
+Parameters args_parse(int argc, char **argv) {
+  Parameters params;
+  unsigned char var_range;
+  bool size_set, slope_set, start_set, var_set, rot_set;
+  int i;
+  char *arg, *value;
 
   params.quiet = false;
   params.seed = (unsigned long)time(NULL);
@@ -128,29 +131,26 @@ parameters parse_args(int argc, char **argv) {
   params.file_path = "output.bmp";
   params.monochrome = false;
 
-  unsigned char var_range = 30;
+  var_range = 30;
 
-  bool size_set = false;
-  bool slope_set = false;
-  bool start_set = false;
-  bool var_set = false;
-  bool rot_set = false;
+  size_set = false;
+  slope_set = false;
+  start_set = false;
+  var_set = false;
+  rot_set = false;
 
-  int i;
-  char *arg;
-  char *value;
   for (i = 1; i < argc; i++) {
     arg = argv[i];
     value = split_arg_value(arg);
     if (is_arg(arg, "--help")) {
-      print_help(0);
+      print_help(EXIT_SUCCESS);
     } else if (is_arg(arg, "-q") || is_arg(arg, "--quiet")) {
       params.quiet = true;
     } else if (is_arg(arg, "-v") || is_arg(arg, "--version")) {
       puts(PACKAGE " " VERSION);
       exit(0);
     } else if (is_arg(arg, "-w") || is_arg(arg, "--width")) {
-      params.width = parse_ushort(arg, value);
+      params.width = parse_uint(arg, value);
       if (params.width == 0) {
         invalid_value(arg, value);
       }
@@ -158,7 +158,7 @@ parameters parse_args(int argc, char **argv) {
         params.height = params.width;
       }
     } else if (is_arg(arg, "-h") || is_arg(arg, "--height")) {
-      params.height = parse_ushort(arg, value);
+      params.height = parse_uint(arg, value);
       if (params.height == 0) {
         invalid_value(arg, value);
       }
@@ -170,13 +170,13 @@ parameters parse_args(int argc, char **argv) {
     } else if (is_arg(arg, "-o") || is_arg(arg, "--output")) {
       params.file_path = value;
     } else if (is_arg(arg, "-p") || is_arg(arg, "--pixel")) {
-      params.size = parse_ushort(arg, value);
+      params.size = parse_uint(arg, value);
       if (params.size == 0) {
         invalid_value(arg, value);
       }
       size_set = true;
     } else if (is_arg(arg, "-s") || is_arg(arg, "--slope")) {
-      params.slope = parse_char(arg, value);
+      params.slope = parse_uint(arg, value);
       slope_set = true;
     } else if (is_arg(arg, "-c") || is_arg(arg, "--color")) {
       parse_color(arg, value, params.start);
@@ -185,9 +185,9 @@ parameters parse_args(int argc, char **argv) {
       parse_color(arg, value, params.var);
       var_set = true;
     } else if (is_arg(arg, "-vr") || is_arg(arg, "--var-range")) {
-      var_range = parse_char(arg, value);
+      var_range = parse_uint(arg, value);
     } else if (is_arg(arg, "-r") || is_arg(arg, "--rotation")) {
-      params.rotation = parse_char(arg, value) % 4;
+      params.rotation = parse_uint(arg, value) % 4;
       rot_set = true;
     } else if (is_arg(arg, "-m") || is_arg(arg, "--monochrome")) {
       params.monochrome = true;
@@ -201,30 +201,30 @@ parameters parse_args(int argc, char **argv) {
     params.height = 1080;
   }
 
-  set_seed(params.seed);
+  rand_seed(params.seed);
 
   if (!size_set) {
-    params.size = rand_ushort(6) + 6;
+    params.size = rand_uint(6) + 6;
   }
 
   if (!slope_set) {
-    params.slope = rand_uchar(50) + 100;
+    params.slope = rand_uint(50) + 100;
   }
 
   if (!start_set) {
-    params.start[0] = rand_uchar(256);
-    params.start[1] = rand_uchar(256);
-    params.start[2] = rand_uchar(256);
+    params.start[0] = rand_uint(256);
+    params.start[1] = rand_uint(256);
+    params.start[2] = rand_uint(256);
   }
 
   if (!var_set) {
-    params.var[0] = rand_uchar(var_range + 1);
-    params.var[1] = rand_uchar(var_range + 1);
-    params.var[2] = rand_uchar(var_range + 1);
+    params.var[0] = rand_uint(var_range + 1);
+    params.var[1] = rand_uint(var_range + 1);
+    params.var[2] = rand_uint(var_range + 1);
   }
 
   if (!rot_set) {
-    params.rotation = rand_uchar(4);
+    params.rotation = rand_uint(4);
   }
 
   return params;

src/args.h

@@ -3,6 +3,6 @@
 #ifndef ARGS_H
 #define ARGS_H
 
-parameters parse_args(int argc, char **argv);
+Parameters args_parse(int argc, char **argv);
 
 #endif

src/bmp.c

@@ -1,37 +1,45 @@
-#include "bmp.h"
 #include <stdio.h>
 #include <stdlib.h>
 
+#include "bmp.h"
+#include "types.h"
+
 #define HEADER_SIZE 54
 
-void write_str(unsigned char *buffer, unsigned int offset, unsigned int size,
-               unsigned char *value) {
+static void write_str(unsigned char *buffer, unsigned int offset,
+                      unsigned int size, unsigned char *value) {
   unsigned int i;
+
   for (i = 0; i < size; i++) {
     buffer[offset + i] = (unsigned char)value[i];
   }
 }
 
-void write_num(unsigned char *buffer, unsigned int offset, unsigned int size,
-               unsigned int value) {
+static void write_num(unsigned char *buffer, unsigned int offset,
+                      unsigned int size, unsigned int value) {
   unsigned int i;
+
   for (i = 0; i < size; i++) {
     buffer[offset + i] = (unsigned char)((value >> (8 * i)) & 0xFFu);
   }
 }
 
-void write_nul(unsigned char *buffer, unsigned int offset, unsigned int size) {
+static void write_nul(unsigned char *buffer, unsigned int offset,
+                      unsigned int size) {
   unsigned int i;
+
   for (i = 0; i < size; i++) {
     buffer[offset + i] = 0;
   }
 }
 
-unsigned char *bmp_header(unsigned short width, unsigned short height,
-                          unsigned char color_depth) {
-  unsigned char *output = (unsigned char *)malloc(HEADER_SIZE);
-  unsigned int data_length = ((unsigned int)width) * ((unsigned int)height) *
-                             ((unsigned int)color_depth);
+static unsigned char *bmp_header(unsigned int width, unsigned int height,
+                                 unsigned int color_depth) {
+  unsigned char *output;
+  unsigned int data_length;
+
+  output = malloc(HEADER_SIZE * sizeof(unsigned char));
+  data_length = width * height * color_depth;
   // 0x00(2) BM
   write_str(output, 0x00, 0x02, (unsigned char *)"BM");
   // 0x02(4) file size
@@ -66,41 +74,45 @@ unsigned char *bmp_header(unsigned short width, unsigned short height,
   return output;
 }
 
-unsigned int bmp_data_line_length(unsigned short width,
-                                  unsigned char color_depth) {
-  unsigned int data_length =
-      ((unsigned int)width) * ((unsigned int)color_depth);
-  unsigned int line_offset = data_length % 4;
-  unsigned short line_padding = line_offset > 0 ? 4 - line_offset : 0;
+static unsigned int bmp_data_line_length(unsigned int width,
+                                         unsigned int color_depth) {
+  unsigned int data_length, line_offset, line_padding;
+
+  data_length = width * color_depth;
+  line_offset = data_length % 4;
+  line_padding = line_offset > 0 ? 4 - line_offset : 0;
+
   return data_length + line_padding;
 }
 
-void bmp_data_line(unsigned char *buffer, unsigned short width,
-                   unsigned char color_depth, unsigned char *data) {
-  unsigned int data_length =
-      ((unsigned int)width) * ((unsigned int)color_depth);
-  unsigned int line_offset = data_length % 4;
-  unsigned short line_padding = line_offset > 0 ? 4 - line_offset : 0;
+static void bmp_data_line(unsigned char *buffer, unsigned int width,
+                          unsigned int color_depth, unsigned char *data) {
+  unsigned int data_length, line_offset, line_padding;
+
+  data_length = width * color_depth;
+  line_offset = data_length % 4;
+  line_padding = line_offset > 0 ? 4 - line_offset : 0;
   write_str(buffer, 0, data_length, data);
   if (line_padding > 0) {
     write_nul(buffer, width * color_depth, line_padding);
   }
 }
 
-void bmp_generate(unsigned short width, unsigned short height,
-                  unsigned char color_depth, bool descending, char *file_path,
+void bmp_generate(unsigned int width, unsigned int height,
+                  unsigned int color_depth, bool descending, char *file_path,
                   line_fn generate_line) {
   FILE *fptr;
+  unsigned char *header, *data_buffer, *line_buffer;
+  unsigned int data_length, line_length, y;
+
   fptr = fopen(file_path, "w");
-  unsigned char *header = bmp_header(width, height, color_depth);
+  header = bmp_header(width, height, color_depth);
   fwrite(header, HEADER_SIZE, 1, fptr);
   free(header);
-  unsigned int data_length =
-      ((unsigned int)width) * ((unsigned int)color_depth);
-  unsigned int line_length = bmp_data_line_length(width, color_depth);
-  unsigned char *data_buffer = (unsigned char *)malloc(data_length);
-  unsigned char *line_buffer = (unsigned char *)malloc(line_length);
-  unsigned short y;
+  data_length = width * color_depth;
+  line_length = bmp_data_line_length(width, color_depth);
+  data_buffer = malloc(data_length * sizeof(unsigned char));
+  line_buffer = malloc(line_length * sizeof(unsigned char));
   for (y = 0; y < height; y++) {
     generate_line(y, data_buffer, data_length);
     bmp_data_line(line_buffer, width, color_depth, data_buffer);

src/bmp.h

@@ -3,8 +3,8 @@
 #ifndef BMP_H
 #define BMP_H
 
-void bmp_generate(unsigned short width, unsigned short height,
-                  unsigned char color_depth, bool descending, char *file_path,
+void bmp_generate(unsigned int width, unsigned int height,
+                  unsigned int color_depth, bool descending, char *file_path,
                   line_fn generate_line);
 
 #endif

src/generator.c

@@ -1,30 +1,36 @@
-#include "args.h"
-#include "bmp.h"
-#include "config.h"
-#include "rand.h"
 #include <stdio.h>
 #include <stdlib.h>
 #include <time.h>
 
+#include "args.h"
+#include "bmp.h"
+#include "config.h"
+#include "rand.h"
+
 #define BMP_COLOR_DEPTH 3
 
-parameters global_params;
-float slope;
-unsigned char color_depth;
-unsigned int line_width;
-unsigned char *last_line;
-unsigned char *current_line;
+static Parameters global_params;
+static float slope;
+static unsigned char color_depth;
+static unsigned int line_width;
+static unsigned char *last_line;
+static unsigned char *current_line;
+
+static unsigned char generate_pixel(unsigned char depth,
+                                    unsigned char top_pixel,
+                                    unsigned char left_pixel) {
+  int k, v;
+
+  k = rand_uint(global_params.var[depth] + 1);
+  v = (rand_uint(2) == 0 ? k : -k) + (left_pixel)*slope +
+      (top_pixel) * (1.0 - slope);
 
-unsigned char generate_pixel(unsigned char depth, unsigned char top_pixel,
-                             unsigned char left_pixel) {
-  short k = rand_uchar(global_params.var[depth] + 1);
-  short v = (rand_uchar(2) == 0 ? k : -k) + (left_pixel)*slope +
-            (top_pixel) * (1.0 - slope);
   return (unsigned char)(v < 0 ? 0 : (v > 255 ? (unsigned char)255 : v));
 }
 
-void generate_line() {
+static void generate_line() {
   unsigned int i;
+
   for (i = 0; i < line_width; i++) {
     last_line[i] = current_line[i];
   }
@@ -39,10 +45,11 @@ void generate_line() {
   }
 }
 
-void generate_bmp_line(unsigned short y, unsigned char *data_buffer,
-                       unsigned int len) {
+static void generate_bmp_line(unsigned int y, unsigned char *data_buffer,
+                              unsigned int len) {
   unsigned int i;
   unsigned int x;
+
   if (y % global_params.size == 0) {
     generate_line();
   }
@@ -53,7 +60,7 @@ void generate_bmp_line(unsigned short y, unsigned char *data_buffer,
   }
 }
 
-void debug_parameters(parameters params) {
+static void debug_parameters(Parameters params) {
   if (!params.quiet) {
     printf("  output  %s\n", params.file_path);
     printf("  seed    %ld\n", params.seed);
@@ -74,38 +81,41 @@ void debug_parameters(parameters params) {
   }
 }
 
-void init(parameters params) {
+static void init(Parameters params) {
+  unsigned int i;
+
   global_params = params;
   slope = ((float)params.slope) / 255.0;
   color_depth = params.monochrome ? 1 : 3;
   debug_parameters(params);
   line_width = ((params.width / params.size) + 1) * color_depth;
-  last_line = (unsigned char *)malloc(line_width);
-  current_line = (unsigned char *)malloc(line_width);
-  unsigned int i = 0;
+  last_line = malloc(line_width * sizeof(unsigned char));
+  current_line = malloc(line_width * sizeof(unsigned char));
   for (i = 0; i < line_width; i++) {
     current_line[i] = params.start[i % color_depth];
   }
-  set_seed(params.seed);
+  rand_seed(params.seed);
 }
 
-void clean() {
+static void clean() {
   free(last_line);
   free(current_line);
 }
 
-void print_time(parameters params, clock_t start) {
+static void print_time(Parameters params, clock_t start) {
   if (!params.quiet) {
     clock_t now = clock();
     printf("time: %.3fs\n", (float)(now - start) / CLOCKS_PER_SEC);
   }
 }
 
-void generate(parameters params) {
+void generator_run(Parameters params) {
+  clock_t start;
+
   if (!params.quiet) {
     puts(PACKAGE " " VERSION);
   }
-  clock_t start = clock();
+  start = clock();
   init(params);
   bmp_generate(params.width, params.height, BMP_COLOR_DEPTH,
                params.rotation % 2 == 1, params.file_path, generate_bmp_line);

src/generator.h

@@ -3,6 +3,6 @@
 #ifndef GENERATOR_H
 #define GENERATOR_H
 
-void generate(parameters params);
+void generator_run(Parameters params);
 
 #endif

src/main.c

@@ -1,9 +1,15 @@
+#include <stdlib.h>
+
 #include "args.h"
 #include "generator.h"
+#include "main.h"
+#include "types.h"
 
 int main(int argc, char **argv) {
-  parameters params;
-  params = parse_args(argc, argv);
-  generate(params);
-  return 0;
+  Parameters params;
+
+  params = args_parse(argc, argv);
+  generator_run(params);
+
+  return EXIT_SUCCESS;
 }

src/main.h

@@ -1,4 +1,6 @@
 #ifndef MAIN_H
 #define MAIN_H
 
+int main(int argc, char **argv);
+
 #endif

src/rand.c

@@ -1,25 +1,27 @@
+#include "rand.h"
+
 static unsigned long long mcg_state = 0xcafef00dd15ea5e5u; // Must be odd
 static unsigned long long const multiplier = 6364136223846793005u;
 
 // https://en.wikipedia.org/wiki/Permuted_congruential_generator
-unsigned long rand(void) {
-  unsigned long long x = mcg_state;
-  unsigned count = (unsigned)(x >> 61);
+static unsigned long rand(void) {
+  unsigned long long x;
+  unsigned count;
+
+  x = mcg_state;
+  count = (unsigned)(x >> 61);
 
   mcg_state = x * multiplier;
   x ^= x >> 22;
+
   return (unsigned long)(x >> (22 + count));
 }
 
-void set_seed(unsigned long long seed) {
+void rand_seed(unsigned long long seed) {
   mcg_state = 2 * seed + 1;
   (void)rand();
 }
 
-unsigned char rand_uchar(const unsigned int max) {
-  return max == 0 ? 0 : (unsigned char)(rand() % max);
-}
-
-unsigned short rand_ushort(const unsigned int max) {
-  return max == 0 ? 0 : (unsigned short)(rand() % max);
+unsigned int rand_uint(unsigned int max) {
+  return max == 0 ? 0 : (unsigned int)(rand() % max);
 }

src/rand.h

@@ -1,8 +1,7 @@
 #ifndef RAND_H
 #define RAND_H
 
-void set_seed(unsigned long long seed);
-unsigned char rand_uchar(unsigned int max);
-unsigned short rand_ushort(unsigned int max);
+void rand_seed(unsigned long long seed);
+unsigned int rand_uint(unsigned int max);
 
 #endif

src/types.h

@@ -3,23 +3,21 @@
 #ifndef TYPES_H
 #define TYPES_H
 
-struct Parameters {
+typedef struct Parameters {
   bool quiet;
   bool monochrome;
   unsigned long seed;
-  unsigned short width;
-  unsigned short height;
+  unsigned int width;
+  unsigned int height;
   char *file_path;
-  unsigned short size;
-  unsigned char slope;
-  unsigned char start[3];
-  unsigned char var[3];
-  unsigned char rotation;
-};
+  unsigned int size;
+  unsigned int slope;
+  unsigned int start[3];
+  unsigned int var[3];
+  unsigned int rotation;
+} Parameters;
 
-typedef struct Parameters parameters;
-
-typedef void line_fn(unsigned short y, unsigned char *data_buffer,
+typedef void line_fn(unsigned int y, unsigned char *data_buffer,
                      unsigned int len);
 
 #endif