From: Simon Ruderich <simon@ruderich.org>
Date: Wed, 1 Jan 2014 15:37:48 +0000 (+0100)
Subject: Fix error_at_line() not always exiting if status != 0.
X-Git-Tag: 0.2~12
X-Git-Url: https://ruderich.org/simon/gitweb/?a=commitdiff_plain;h=39d313d1d474dbe025270b9555a5dfb03286c8c9;p=coloredstderr%2Fcoloredstderr.git

Fix error_at_line() not always exiting if status != 0.

Also add tests.
---

diff --git a/NEWS b/NEWS
index 2c5e477..9c37519 100644
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,14 @@
 NEWS
 ====
 
+0.X
+---
+
+- Fix hook for GNU's error_at_line() not exiting if status != 0. If
+  error_one_per_line was used and suppressed a message, then error_at_line()
+  didn't exit.
+
+
 0.1
 ---
 
diff --git a/src/coloredstderr.c b/src/coloredstderr.c
index e6f1f6a..2970a1e 100644
--- a/src/coloredstderr.c
+++ b/src/coloredstderr.c
@@ -371,7 +371,7 @@ static void error_vararg(int status, int errnum,
     if (error_one_per_line
             && filename != NULL && linenum != 0
             && filename == last_filename && linenum == last_linenum) {
-        return;
+        goto out;
     }
     last_filename = filename;
     last_linenum  = linenum;
@@ -404,6 +404,7 @@ static void error_vararg(int status, int errnum,
 
     fprintf(stderr, "\n");
 
+out:
     if (status != 0) {
         exit(status);
     }
diff --git a/tests/example_error.c b/tests/example_error.c
index 0ed8746..0d979db 100644
--- a/tests/example_error.c
+++ b/tests/example_error.c
@@ -20,12 +20,16 @@
 #include <config.h>
 
 #define _GNU_SOURCE /* for program_invocation_name */
-#include <stdio.h>
-#include <stdlib.h>
 #include <errno.h>
 #include <error.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <unistd.h>
 
 #include "../src/compiler.h"
+#include "example.h"
 
 
 void (*error_print_progname)(void);
@@ -37,26 +41,41 @@ static void print_progname(void) {
 
 
 int main(int argc unused, char **argv unused) {
-    program_invocation_name = "./example_error";
+    pid_t pid;
+
+    char name[] = "./example_error";
+    program_invocation_name = name;
 
     error(0, 0, "<message>");
     error_at_line(0, 0, "file", 42, "<message>");
+    FORKED_TEST(pid) { error(1, 0, "<message>"); }
+    FORKED_TEST(pid) { error_at_line(1, 0, "file", 42, "<message>"); }
 
     error(0, ENOMEM, "<message>");
     error_at_line(0, ENOMEM, "file", 42, "<message>");
     error_at_line(0, ENOMEM, "file", 42, "<message>");
+    FORKED_TEST(pid) { error(1, ENOMEM, "<message>"); }
+    FORKED_TEST(pid) { error_at_line(1, ENOMEM, "file", 42, "<message>"); }
+
+    fflush(stdout);
+    printf("\n\n");
+    fflush(stdout);
 
     error_print_progname = print_progname;
     error_one_per_line = 1;
 
     error(0, 0, "<message>");
     error_at_line(0, 0, "file", 42, "<message>");
+    FORKED_TEST(pid) { error(1, 0, "<message>"); }
+    FORKED_TEST(pid) { error_at_line(1, 0, "file", 42, "<message>"); }
 
     error(0, ENOMEM, "<message>");
     error_at_line(0, ENOMEM, "file", 42, "<message>");
     error_at_line(0, ENOMEM, "file", 42, "<message>");
+    FORKED_TEST(pid) { error(1, ENOMEM, "<message>"); }
 
-    /* Exit codes are not tested. */
+    error_one_per_line = 0;
+    FORKED_TEST(pid) { error_at_line(1, ENOMEM, "file", 42, "<message>"); }
 
     return EXIT_SUCCESS;
 }
diff --git a/tests/example_error.expected b/tests/example_error.expected
index 4302575..01c76b8 100644
--- a/tests/example_error.expected
+++ b/tests/example_error.expected
@@ -1,10 +1,27 @@
 >STDERR>./example_error: <message>
 ./example_error:file:42: <message>
-./example_error: <message>: Cannot allocate memory
+./example_error: <message>
+<STDERR<exit code: 1
+>STDERR>./example_error:file:42: <message>
+<STDERR<exit code: 1
+>STDERR>./example_error: <message>: Cannot allocate memory
 ./example_error:file:42: <message>: Cannot allocate memory
 ./example_error:file:42: <message>: Cannot allocate memory
-PROG<message>
+./example_error: <message>: Cannot allocate memory
+<STDERR<exit code: 1
+>STDERR>./example_error:file:42: <message>: Cannot allocate memory
+<STDERR<exit code: 1
+
+
+>STDERR>PROG<message>
 PROGfile:42: <message>
-PROG<message>: Cannot allocate memory
+PROG<message>
+<STDERR<exit code: 1
+exit code: 1
+>STDERR>PROG<message>: Cannot allocate memory
 PROGfile:42: <message>: Cannot allocate memory
-<STDERR<EOF
+PROG<message>: Cannot allocate memory
+<STDERR<exit code: 1
+>STDERR>PROGfile:42: <message>: Cannot allocate memory
+<STDERR<exit code: 1
+EOF