warnings: refactoring

The code here was too confusing, this seems more natural.

* src/complain.c (error_message): Move the indentation check and the category
output to complains. Also, no longer take a 'warnings' argument.
(complains): Factor calls to error_message.

Signed-off-by: Akim Demaille <akim@lrde.epita.fr>

diff --git a/src/complain.c b/src/complain.c
index c0de06d7ab0f63c50b260680621f3cd912c616f1..926d90139174ba6526d9ee0eae68e62850c0a197 100644 (file)
--- a/src/complain.c
+++ b/src/complain.c
@@ -84,7 +84,6 @@ void
 error_message (const location *loc, warnings flags, const char *prefix,
                const char *message, va_list args)
 {
 error_message (const location *loc, warnings flags, const char *prefix,
                const char *message, va_list args)
 {

-  (void) flags;

   unsigned pos = 0;
 
   if (loc)
   unsigned pos = 0;
 
   if (loc)


@@ -95,6 +94,8 @@ error_message (const location *loc, warnings flags, const char *prefix,
 
   if (indent_ptr)
     {
 
   if (indent_ptr)
     {

+      if (*indent_ptr)
+        prefix = NULL;

       if (!*indent_ptr)
         *indent_ptr = pos;
       else if (*indent_ptr > pos)
       if (!*indent_ptr)
         *indent_ptr = pos;
       else if (*indent_ptr > pos)


@@ -125,21 +126,16 @@ complains (const location *loc, warnings flags, const char *message,
            va_list args)
 
 {
            va_list args)
 
 {

+  const char* prefix =
+    flags & fatal ? _("fatal error")
+    : flags & (errors_flag | complaint) ? _("error")
+    : _("warning");
+
+  complaint_issued |= flags & (complaint | errors_flag);
+  if (flags & (warnings_flag | silent | fatal | complaint))
+    error_message (loc, flags, prefix, message, args);

   if (flags & fatal)
   if (flags & fatal)

-    {
-       error_message (loc, fatal, _("fatal error"), message, args);
-       exit (EXIT_FAILURE);
-    }
-  else if (flags & (complaint | warnings_flag | silent))
-    {
-      const char* prefix =
-        flags & (errors_flag | complaint) ? _("error") : _("warning");
-      if (flags & (complaint | errors_flag))
-        complaint_issued = true;
-      error_message (loc, flags,
-                     indent_ptr && *indent_ptr ? NULL : prefix,
-                     message, args);
-    }
+    exit (EXIT_FAILURE);

 }
 
 void
 }
 
 void