commit 225068fc5a274de0177d09066001a492ec1136ac
author Nigel Tao <nigeltao@golang.org> Sun Sep 15 10:08:28 2024 +1000
committer Nigel Tao <nigeltao@golang.org> Sun Sep 15 10:08:28 2024 +1000
tree d790fac5c87e387a7a77310f866c0509ae3ddf74
parent 3387e19512ba52957f29fa3a357f11360b415b08

stb-imagedumper: handle resize 0w or 0h gracefully

example/stb-imagedumper/stb-imagedumper.c

diff --git a/example/stb-imagedumper/stb-imagedumper.c b/example/stb-imagedumper/stb-imagedumper.c
index 21eda11..31d3d5e 100644
--- a/example/stb-imagedumper/stb-imagedumper.c
+++ b/example/stb-imagedumper/stb-imagedumper.c
@@ -839,7 +839,11 @@
 
   char resize_info_string[64];
   resize_info_string[0] = 0;
-  if (g_flags.resize > 0) {
+  if (g_flags.resize <= 0) {
+    // No-op.
+  } else if ((src_w <= 0) || (src_h <= 0)) {
+    snprintf(resize_info_string, sizeof(resize_info_string), " → 0×0");
+  } else {
     int resize_w = src_w;
     int resize_h = src_h;
     if (!resize_wh(&resize_w, &resize_h, g_flags.resize)) {
@@ -854,15 +858,16 @@
          filename, filesize, src_w, src_h, resize_info_string, elapsed_micros);
 
   if (!src_pixels) {
-    printf("%s\n", stbi_failure_reason());
+    printf("%s\n\n", stbi_failure_reason());
     return;
   } else if ((src_w < 0) || (MAX_INCL_DIMENSION < src_w) ||  //
              (src_h < 0) || (MAX_INCL_DIMENSION < src_h)) {
-    printf("main: image is too large\n");
+    printf("main: image is too large\n\n");
     stbi_image_free(src_pixels);
     return;
   } else if ((src_w == 0) || (src_h == 0)) {
     stbi_image_free(src_pixels);
+    printf("\n");
     return;
   }
 
@@ -874,6 +879,7 @@
     dst_pixels = resize(&w, &h, src_pixels, g_flags.resize, bytes_per_pixel);
     if (!dst_pixels) {
       stbi_image_free(src_pixels);
+      printf("\n");
       return;
     }
     pixels = dst_pixels;