6 files changed, 24 insertions(+), 2 deletions(-)

M image-viewer/include/image.h
M image-viewer/src/image.c
M image-viewer/src/main.c
M text-viewer/include/text_viewer.h
M text-viewer/src/main.c
M text-viewer/src/text_viewer.c

@@ 12,6 12,7 @@
 #include <stdint.h>
 
 #include "direction.h"
+#include "logger.h"
 #include "display_utils.h"
 
 typedef enum {


@@ 63,6 64,8 @@ typedef struct {
  */
 image_t image_create(char *path);
 
+void image_error_log(logger_t *logger, image_error_t error);
+
 /**
  * @brief Clean up image data
  *

@@ 1,9 1,17 @@
 #include "image.h"
 #include "direction.h"
 #include "display_utils.h"
+#include "logger.h"
 #include <stdlib.h>
 
+char *image_error_strings[] = {
+    "Success",        "Unknown format",   "No such file or directory",
+    "No permissions", "Cannot open file", "Wrong format",
+    "Unknown"};
 
+void image_error_log(logger_t *logger, image_error_t error) {
+  logger_error(logger, __FILE__, __FUNCTION__, __LINE__, "There was an error loading image: %s", image_error_strings[error]);
+}
 
 image_t image_create(char *path) {
   image_t image = {

@@ 82,7 82,7 @@ int main(int argc, char *argv[])
   rgb_led_clear(&led, LED_RIGHT);
 
   if (viewer.error != IMERR_SUCCESS) {
-    logger_error(&logger, __FILE__, __FUNCTION__, __LINE__, "Could not load image %d", viewer.error);
+    image_error_log(&logger, viewer.error);
     rgb_led_set_red(&led, LED_RIGHT);
     return IMAGE_ERROR;
   }

@@ 39,6 39,8 @@ typedef struct {
 text_viewer_t text_viewer_create(char *path, mzapo_pheripherals_t pheripherals,
                                  logger_t *logger, font_t font);
 
+void file_error_log(logger_t *logger, file_error_t error);
+
 /**
  * @brief Cleans up text viewer data
  *

@@ 75,7 75,7 @@ int main(int argc, char *argv[]) {
               "Loading file");
   file_error_t error = text_viewer_load_file(&text_viewer);
   if (error != FILER_SUCCESS) {
-    logger_error(&logger, __FILE__, __FUNCTION__, __LINE__, "Could not load file.");
+    file_error_log(&logger, error);
     rgb_led_set_red(&rgb_leds, LED_RIGHT);
     return ERROR_CANT_OPEN_FILE;
   }

@@ 7,6 7,7 @@
 #include "gui_component_text_view.h"
 #include "input.h"
 #include "keyboard_const.h"
+#include "logger.h"
 #include "mzapo_led_strip.h"
 #include "mzapo_rgb_led.h"
 #include "renderer.h"


@@ 17,6 18,14 @@
 #include <stdlib.h>
 #include <string.h>
 
+char *file_error_strings[] = {"Success",          "No such file or directory",
+                              "No permissions",   "Cannot open file",
+                              "Cannot read file", "Unknown"};
+
+void file_error_log(logger_t *logger, file_error_t error) {
+  logger_error(logger, __FILE__, __FUNCTION__, __LINE__, "There was an error loading file: %s", file_error_strings[error]);
+}
+
 text_viewer_t text_viewer_create(char *path, mzapo_pheripherals_t pheripherals,
                                  logger_t *logger, font_t font) {
   text_viewer_t text_viewer = {.pheripherals = pheripherals,