check_all_msg

diff --git a/recvcmd.c b/recvcmd.c
index e9d37d834f789f31aa23fbfa5949da4e0f33b591..ac5bbe359312f28488a2d2dd1e475ba54dd83bd6 100644 (file)
--- a/recvcmd.c
+++ b/recvcmd.c
@@ -39,13 +39,13 @@ static short check_msg (BODY * b, short err)
   return 0;
 }
 
-static short check_all_msg (ATTACHPTR ** idx, short idxlen,
-                           BODY * cur, short err)
+static bool check_all_msg (ATTACHPTR ** idx, short idxlen,
+                           BODY * cur, bool err)
 {
   short i;
 
   if (cur && check_msg (cur, err) == -1)
-    return -1;
+    return false;
   else if (!cur)
   {
     for (i = 0; i < idxlen; i++)
@@ -53,11 +53,11 @@ static short check_all_msg (ATTACHPTR ** idx, short idxlen,
       if (idx[i]->content->tagged)
       {
        if (check_msg (idx[i]->content, err) == -1)
-         return -1;
+         return false;
       }
     }
   }
-  return 0;
+  return true;
 }
 
 
@@ -123,7 +123,7 @@ void mutt_attach_bounce (FILE * fp, HEADER * hdr,
   int ret = 0;
   int p   = 0;
 
-  if (check_all_msg (idx, idxlen, cur, 1) == -1)
+  if (!check_all_msg (idx, idxlen, cur, true))
     return;
 
   /* one or more messages? */
@@ -244,7 +244,7 @@ void mutt_attach_resend (FILE * fp, HEADER * hdr, ATTACHPTR ** idx,
 {
   short i;
 
-  if (check_all_msg (idx, idxlen, cur, 1) == -1)
+  if (!check_all_msg (idx, idxlen, cur, true))
     return;
 
   if (cur)
@@ -673,7 +673,7 @@ void mutt_attach_forward (FILE * fp, HEADER * hdr,
   short nattach;
 
 
-  if (check_all_msg (idx, idxlen, cur, 0) == 0)
+  if (check_all_msg (idx, idxlen, cur, false))
     attach_forward_msgs (fp, hdr, idx, idxlen, cur, flags);
   else
   {
@@ -833,7 +833,7 @@ void mutt_attach_reply (FILE * fp, HEADER * hdr,
     unset_option (OPTNEWSSEND);
 #endif
 
-  if (check_all_msg (idx, idxlen, cur, 0) == -1)
+  if (!check_all_msg (idx, idxlen, cur, false))
   {
     nattach = count_tagged (idx, idxlen);
     if ((parent = find_parent (idx, idxlen, cur, nattach)) == NULL)