From a6ac47fb763fd4448c0269e274f061226fa0f10a Mon Sep 17 00:00:00 2001
From: Phil Jones <philj56@gmail.com>
Date: Fri, 10 Jun 2022 18:25:43 +0100
Subject: Minor cleanup.

---
 src/entry_backend/harfbuzz.c | 28 ++++++++++++++++++----------
 src/entry_backend/harfbuzz.h |  1 -
 src/log.c                    |  1 -
 3 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/src/entry_backend/harfbuzz.c b/src/entry_backend/harfbuzz.c
index 161cefa..78d0757 100644
--- a/src/entry_backend/harfbuzz.c
+++ b/src/entry_backend/harfbuzz.c
@@ -1,10 +1,7 @@
 #include <cairo/cairo.h>
-#include <glib.h>
 #include <harfbuzz/hb-ft.h>
 #include <harfbuzz/hb-glib.h>
 #include <math.h>
-#include <pango/pangocairo.h>
-#include <pango/pango.h>
 #include <wchar.h>
 #include "harfbuzz.h"
 #include "../entry.h"
@@ -28,6 +25,17 @@ const struct {
 
 #include <freetype/fterrors.h>
 
+static const char *get_ft_error_string(int err_code)
+{
+	for (size_t i = 0; i < N_ELEM(ft_errors); i++) {
+		if (ft_errors[i].err_code == err_code) {
+			return ft_errors[i].err_msg;
+		}
+	}
+
+	return "Unknown FT error";
+}
+
 /*
  * Cairo / FreeType use 72 Pts per inch, but Pango uses 96 DPI, so we have to
  * rescale for consistency.
@@ -110,7 +118,7 @@ void entry_backend_init(
 	err = FT_Init_FreeType(&entry->backend.ft_library);
 	if (err) {
 		log_error("Error initialising FreeType: %s\n",
-				ft_errors[err].err_msg);
+				get_ft_error_string(err));
 		exit(EXIT_FAILURE);
 	}
 
@@ -119,18 +127,18 @@ void entry_backend_init(
 			entry->backend.ft_library, "font.ttf", 0,
 			&entry->backend.ft_face);
 	if (err) {
-		log_error("Error loading font: %s\n", ft_errors[err].err_msg);
+		log_error("Error loading font: %s\n", get_ft_error_string(err));
 		exit(EXIT_FAILURE);
 	}
 	err = FT_Set_Char_Size(
 			entry->backend.ft_face,
-			entry->font_size * 64 * scale,
-			entry->font_size * 64 * scale,
-			96,
-			96);
+			entry->font_size * 64,
+			entry->font_size * 64,
+			96 * scale,
+			96 * scale);
 	if (err) {
 		log_error("Error setting font size: %s\n",
-				ft_errors[err].err_msg);
+				get_ft_error_string(err));
 		exit(EXIT_FAILURE);
 	}
 
diff --git a/src/entry_backend/harfbuzz.h b/src/entry_backend/harfbuzz.h
index 52116a8..499e973 100644
--- a/src/entry_backend/harfbuzz.h
+++ b/src/entry_backend/harfbuzz.h
@@ -5,7 +5,6 @@
 #include <ft2build.h>
 #include FT_FREETYPE_H
 #include <harfbuzz/hb.h>
-#include <pango/pangocairo.h>
 
 struct entry;
 
diff --git a/src/log.c b/src/log.c
index 2d7e857..97dcdbe 100644
--- a/src/log.c
+++ b/src/log.c
@@ -36,7 +36,6 @@ void log_error(const char *const fmt, ...)
 	va_list args;
 	va_start(args, fmt);
 	fprintf(stderr, "[ERROR]: ");
-	print_indent(stderr);
 	vfprintf(stderr, fmt, args);
 	va_end(args);
 }
-- 
cgit v1.2.3