meson.build

@@ -10,7 +10,7 @@ libtoml_dep = dependency(
 
 exe = executable(
 	'smeargle',
-	['src/main.c', 'src/font.c'],
+	'src/main.c',
 	dependencies: libtoml_dep,
 	install: true
 )

src/font.c

@@ -4,29 +4,25 @@
 #include "util.h"
 #include "font.h"
 
-font_t font_create(char *font_name, char *filename) {
+font_t font_create(char *name, char *filename) {
 	FILE *fp;
 	font_t font;
-	char errbuf[200];
-	map_t *map;
 
-	printf("Processing %s...\n", font_name);
+	printf("Processing %s...\n", name);
 
 	fp = fopen(filename, "r");
 
 	if (!fp) {
-		char str[200];
+		char errbuf[200];
-		snprintf(str, 200, "can't open %s: ", filename);
+		snprintf(errbuf, 200, "can't open %s: ", filename);
-		error(str, strerror(errno));
+		error(errbuf, strerror(errno));
 	}
-	toml_table_t *table = toml_parse_file(fp, errbuf, 200);
+	toml_table_t *table = toml_parse_file(fp);
 	fclose(fp);
 
-#define TOML_GET_KEY(var, key, func) toml_datum_t (var) = func(table, (key));\
+#define TOML_GET_KEY((var, key, func)) toml_datum_t *(var) = func(table, key);\
-	if (!(var).ok) {\
+	if (!var.ok) {\
-		char str[200];\
+		error("key '" key "' missing from", filename); \
-		snprintf(str, 200, "key '%s' missing from", (key)); \
-		error(str, filename); \
 	}
 
 	TOML_GET_KEY(name, "name", toml_string_in)
@@ -35,25 +31,21 @@ font_t font_create(char *font_name, char *filename) {
 	TOML_GET_KEY(width, "width", toml_int_in)
 	TOML_GET_KEY(height, "height", toml_int_in)
 #undef TOML_GET_KEY
-	toml_table_t *mapping = toml_table_in(table, "map");
+	toml_table_t *mapping = toml_table_in(table, "map")
+	if (!mapping.ok) {
+		error("map missing for font ", font);
+	}
 
 	font.name = name.u.s;
-	font.toml_filename = image_filename.u.s;
+	font.filename = image_filename.u.s;
-	font.bits_per_pixel = bpp.u.i;
+	font.bits_per_pixel = bpp;
-	font.width = width.u.i;
+	font.width = width;
-	font.height = height.u.i;
+	font.height = height;
-	font.map_entries = map_create(mapping, &map);
+	font.map_entries = map_create(mapping, *map);
-	font.map = map;
 
 	return font;
 }
 
-void font_destroy(font_t font) {
-	map_destroy(font.map);
-	free(font.name);
-	free(font.toml_filename);
-}
-
 unsigned int map_create(toml_table_t *table, map_t **map) {
 	map_t *node;
 	map_t *next;
@@ -63,15 +55,15 @@ unsigned int map_create(toml_table_t *table, map_t **map) {
 	toml_datum_t spcidx = toml_int_at(space, 0);
 	toml_datum_t spcwid = toml_int_at(space, 1);
 	if (!spcidx.ok) {
-		error("missing or inappropriate index for glyph: ' '", "");
+		error("missing or inappropriate index for glyph: ", key);
 	}
 	if (!spcwid.ok) {
-		error("missing or inappropriate index for glyph: ' '", "");
+		error("missing or inappropriate index for glyph: ", key);
 	}
 	node = root;
 
 	for (nodes = 1; ; nodes++) {
-		const char *key = toml_key_in(table, nodes);
+		char *key = toml_key_in(table, nodes);
 		if (!key) break;
 		toml_array_t *val = toml_array_in(table, key);
 		toml_datum_t index = toml_int_at(val, 0);
@@ -90,18 +82,4 @@ unsigned int map_create(toml_table_t *table, map_t **map) {
 		node->next = next;
 		node = next;
 	}
-	node->next = NULL;
-
-	*map = root;
-	return nodes;
-}
-
-void map_destroy(map_t *map) {
-	map_t *node = map;
-	map_t *next;
-
-	while ((next = node->next) != NULL) {
-		free(node);
-		node = next;
-	}
 }

src/font.h

@@ -1,7 +1,7 @@
 
 typedef struct map_t {
-	const char *glyph;
+	char *glyph;
-	unsigned int index;
+	unsigned int pos;
 	unsigned int width;
 	struct map_t *next;
 } map_t;
@@ -15,8 +15,3 @@ typedef struct {
 	unsigned int map_entries;
 	map_t *map;
 } font_t;
-
-font_t font_create(char *name, char *filename);
-void font_destroy(font_t font);
-unsigned int map_create(toml_table_t *table, map_t **map);
-void map_destroy(map_t *map);

src/main.c

@@ -6,8 +6,6 @@
 
 #include <toml.h>
 
-#include "util.h"
-
 #define ERRBUFSZ 200
 
 int main(int argc, char **argv) {

src/util.h

@@ -1,11 +1,7 @@
 
-#include <sys/errno.h>
-#include <stdlib.h>
-#include <string.h>
 #include <stdio.h>
 
 static void error(const char *msg, const char *msg1) {
 	fprintf(stderr, "Error: %s%s\n", msg, msg1 ? msg1 : "");
 	exit(1);
 }
-