well this makes things fail for people ...
[apps/madmutt.git] / commands.c
index beaac30..f15ca1c 100644 (file)
@@ -433,7 +433,7 @@ int mutt_select_sort (int reverse)
   if (reverse)
     Sort |= SORT_REVERSE;
 
-  return (Sort != method ? 0 : -1);     /* no need to resort if it's the same */
+  return Sort != method ? 0 : -1;     /* no need to resort if it's the same */
 }
 
 /* invoke a command in a subshell */
@@ -557,7 +557,7 @@ int _mutt_save_message (HEADER * h, CONTEXT * ctx, int delete, int decode,
       mutt_set_flag (Context, h, M_TAG, 0);
     mutt_set_flag (Context, h, M_APPENDED, 1);
   }
-  return (0);
+  return 0;
 }
 
 /* returns 0 if the copy/save was successful, or -1 on error/abort */
@@ -610,7 +610,7 @@ int mutt_save_message (HEADER * h, int delete,
 
   mutt_pretty_mailbox (buf);
   if (mutt_enter_fname (prompt, buf, sizeof (buf), redraw, 0) == -1)
-    return (-1);
+    return -1;
 
   if (*redraw != REDRAW_FULL) {
     if (!h)
@@ -620,7 +620,7 @@ int mutt_save_message (HEADER * h, int delete,
   }
 
   if (!buf[0])
-    return (-1);
+    return -1;
 
   /* This is an undocumented feature of ELM pointed out to me by Felix von
    * Leitner <leitner@prz.fu-berlin.de>
@@ -657,7 +657,7 @@ int mutt_save_message (HEADER * h, int delete,
     if (h) {
       if (_mutt_save_message (h, &ctx, delete, decode, decrypt) != 0) {
         mx_close_mailbox (&ctx, NULL);
-        return (-1);
+        return -1;
       }
     } else {
       for (i = 0; i < Context->vcount; i++) {
@@ -667,7 +667,7 @@ int mutt_save_message (HEADER * h, int delete,
           if (_mutt_save_message (Context->hdrs[Context->v2r[i]], &ctx, delete,
                                   decode, decrypt) != 0) {
             mx_close_mailbox (&ctx, NULL);
-            return (-1);
+            return -1;
           }
         }
       }
@@ -689,7 +689,7 @@ int mutt_save_message (HEADER * h, int delete,
     }
 
     mutt_clear_error ();
-    return (0);
+    return 0;
   }
 
   return -1;