feat(cli): make missing src error msgs consistent

dbohdan · Sep 29, 2023 · 1a77915 · 1a77915
1 parent ebf854b
commit 1a77915
Showing 1 changed file with 29 additions and 12 deletions.
diff --git a/cli.c b/cli.c
@@ -67,6 +67,21 @@ cp_image_t rgb_to_cp(const hicolor_metadata meta, const hicolor_rgb* rgb_img)
     return img;
 }
 
+bool check_src_exists(
+    const char* src
+) {
+    if (access(src, F_OK) != 0) {
+        fprintf(
+            stderr,
+            HICOLOR_CLI_ERROR "source image \"%s\" doesn't exist\n",
+            src
+        );
+        return false;
+    }
+
+    return true;
+}
+
 bool png_to_hicolor(
     hicolor_version version,
     bool dither,
@@ -76,12 +91,8 @@ bool png_to_hicolor(
 {
     hicolor_result res;
 
-    if (access(src, F_OK) != 0) {
-        fprintf(
-            stderr,
-            HICOLOR_CLI_ERROR "source image \"%s\" doesn't exist\n",
-            src
-        );
+    bool exists = check_src_exists(src);
+    if (!exists) {
         return false;
     }
 
@@ -159,12 +170,8 @@ bool png_quantize(
 {
     hicolor_result res;
 
-    if (access(src, F_OK) != 0) {
-        fprintf(
-            stderr,
-            HICOLOR_CLI_ERROR "source image \"%s\" doesn't exist\n",
-            src
-        );
+    bool exists = check_src_exists(src);
+    if (!exists) {
         return false;
     }
 
@@ -231,6 +238,11 @@ bool hicolor_to_png(
 {
     hicolor_result res;
 
+    bool exists = check_src_exists(src);
+    if (!exists) {
+        return false;
+    }
+
     FILE* hi_file = fopen(src, "rb");
     if (hi_file == NULL) {
         fprintf(
@@ -284,6 +296,11 @@ bool hicolor_print_info(
 {
     hicolor_result res;
 
+    bool exists = check_src_exists(src);
+    if (!exists) {
+        return false;
+    }
+
     FILE* hi_file = fopen(src, "rb");
     if (hi_file == NULL) {
         fprintf(