]> granicus.if.org Git - neomutt/commitdiff
tidy: remove ()s from return 488/head
authorRichard Russon <rich@flatcap.org>
Sun, 26 Mar 2017 01:19:36 +0000 (02:19 +0100)
committerRichard Russon <rich@flatcap.org>
Thu, 30 Mar 2017 22:19:16 +0000 (23:19 +0100)
The 'return' operator doesn't require its argument to the ()'d.
This has lead to some very inconsistent code.

This commit removes the ()s from return where the argument is a single
token, e.g.
    return 42;
    return fn(x);

There are ~480 occurrences.

60 files changed:
addrbook.c
alias.c
attach.c
browser.c
buffer.c
buffy.c
color.c
commands.c
complete.c
compose.c
copy.c
crypt-gpgme.c
crypt.c
curs_lib.c
curs_main.c
date.c
edit.c
filter.c
flags.c
from.c
handler.c
hash.c
hdrline.c
help.c
history.c
hook.c
imap/imap.c
init.c
keymap.c
lib.c
mbox.c
menu.c
mh.c
mutt_lua.c
mutt_ssl_gnutls.c
muttlib.c
mx.c
newsrc.c
pager.c
parse.c
pattern.c
pgp.c
pgpinvoke.c
pgpkey.c
postpone.c
query.c
recvattach.c
recvcmd.c
remailer.c
rfc3676.c
rfc822.c
score.c
send.c
sendlib.c
signal.c
smime.c
sort.c
status.c
system.c
thread.c

index b101c876555b48be350717a77a432e5b0bca7f13..50e19fadb6d26fd6f7443356250d56913b1d2bfc 100644 (file)
@@ -72,7 +72,7 @@ alias_format_str (char *dest, size_t destlen, size_t col, int cols, char op, con
       break;
   }
 
-  return (src);
+  return src;
 }
 
 static void alias_entry (char *s, size_t slen, MUTTMENU *m, int num)
diff --git a/alias.c b/alias.c
index a30f81352a0eaf20f9391e7151972b09063f1971..e8a549d8a251787bc00742abd7bc8129c62ce846 100644 (file)
--- a/alias.c
+++ b/alias.c
@@ -34,8 +34,8 @@ ADDRESS *mutt_lookup_alias (const char *s)
 
   for (; t; t = t->next)
     if (mutt_strcasecmp (s, t->name) == 0)
-      return (t->addr);
-  return (NULL);   /* no such alias */
+      return t->addr;
+  return NULL;   /* no such alias */
 }
 
 static ADDRESS *expand_aliases_r (ADDRESS *a, LIST **expn)
@@ -121,7 +121,7 @@ static ADDRESS *expand_aliases_r (ADDRESS *a, LIST **expn)
     rfc822_qualify (head, fqdn);
   }
 
-  return (head);
+  return head;
 }
 
 ADDRESS *mutt_expand_aliases (ADDRESS *a)
index fe848a8204c67eade7496bed87a1b19dc5a23abc..ebf42ad088238a185932eca3c5990c92b3ed3fd6 100644 (file)
--- a/attach.c
+++ b/attach.c
@@ -345,7 +345,7 @@ int mutt_view_attachment (FILE *fp, BODY *a, int flag, HEADER *hdr,
   is_message = mutt_is_message_type(a->type, a->subtype);
   if (WithCrypto && is_message && a->hdr && (a->hdr->security & ENCRYPT) &&
       !crypt_valid_passphrase(a->hdr->security))
-    return (rc);
+    return rc;
   use_mailcap = (flag == MUTT_MAILCAP ||
                (flag == MUTT_REGULAR && mutt_needs_mailcap (a)));
   snprintf (type, sizeof (type), "%s/%s", TYPE (a), a->subtype);
@@ -760,7 +760,7 @@ int mutt_save_attachment (FILE *fp, BODY *m, char *path, int flags, HEADER *hdr)
       {
        mutt_perror ("fopen");
        mutt_sleep (2);
-       return (-1);
+       return -1;
       }
       fseeko ((s.fpin = fp), m->offset, 0);
       mutt_decode_attachment (m, &s);
@@ -769,7 +769,7 @@ int mutt_save_attachment (FILE *fp, BODY *m, char *path, int flags, HEADER *hdr)
       {
        mutt_perror ("fclose");
        mutt_sleep (2);
-       return (-1);
+       return -1;
       }
     }
   }
@@ -782,14 +782,14 @@ int mutt_save_attachment (FILE *fp, BODY *m, char *path, int flags, HEADER *hdr)
     if ((ofp = fopen (m->filename, "r")) == NULL)
     {
       mutt_perror ("fopen");
-      return (-1);
+      return -1;
     }
 
     if ((nfp = save_attachment_open (path, flags)) == NULL)
     {
       mutt_perror ("fopen");
       safe_fclose (&ofp);
-      return (-1);
+      return -1;
     }
 
     if (mutt_copy_stream (ofp, nfp) == -1)
@@ -797,13 +797,13 @@ int mutt_save_attachment (FILE *fp, BODY *m, char *path, int flags, HEADER *hdr)
       mutt_error (_("Write fault!"));
       safe_fclose (&ofp);
       safe_fclose (&nfp);
-      return (-1);
+      return -1;
     }
     safe_fclose (&ofp);
     if (safe_fsync_close (&nfp) != 0)
     {
       mutt_error (_("Write fault!"));
-      return (-1);
+      return -1;
     }
   }
 
@@ -833,7 +833,7 @@ int mutt_decode_save_attachment (FILE *fp, BODY *m, char *path,
   if (s.fpout == NULL)
   {
     mutt_perror ("fopen");
-    return (-1);
+    return -1;
   }
 
   if (fp == NULL)
@@ -846,13 +846,13 @@ int mutt_decode_save_attachment (FILE *fp, BODY *m, char *path,
     {
       mutt_perror ("stat");
       safe_fclose (&s.fpout);
-      return (-1);
+      return -1;
     }
 
     if ((s.fpin = fopen (m->filename, "r")) == NULL)
     {
       mutt_perror ("fopen");
-      return (-1);
+      return -1;
     }
 
     saved_encoding = m->encoding;
@@ -957,7 +957,7 @@ int mutt_print_attachment (FILE *fp, BODY *a)
       {
        mutt_perror ("fopen");
        rfc1524_free_entry (&entry);
-       return (0);
+       return 0;
       }
 
       if ((thepid = mutt_create_filter (command, &fpout, NULL, NULL)) < 0)
@@ -985,7 +985,7 @@ int mutt_print_attachment (FILE *fp, BODY *a)
       unlink(newfile);
 
     rfc1524_free_entry (&entry);
-    return (1);
+    return 1;
   }
 
   if ((ascii_strcasecmp ("text/plain", type) == 0) ||
index 80d038c1f24407d6996c399dab00ff184f1ff617..12075c7b62f6df36427374164916f6b801a6d90a 100644 (file)
--- a/browser.c
+++ b/browser.c
@@ -441,7 +441,7 @@ folder_format_str (char *dest, size_t destlen, size_t col, int cols, char op, co
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, folder_format_str, data, 0);
 
-  return (src);
+  return src;
 }
 
 #ifdef USE_NNTP
@@ -543,7 +543,7 @@ newsgroup_format_str (char *dest, size_t destlen, size_t col, int cols, char op,
       }
       break;
   }
-  return (src);
+  return src;
 }
 #endif /* USE_NNTP */
 
@@ -656,13 +656,13 @@ static int examine_directory (MUTTMENU *menu, struct browser_state *state,
       }
     }
     mutt_perror (d);
-    return (-1);
+    return -1;
   }
 
   if (!S_ISDIR (s.st_mode))
   {
     mutt_error (_("%s is not a directory."), d);
-    return (-1);
+    return -1;
   }
 
   mutt_buffy_check (0);
@@ -670,7 +670,7 @@ static int examine_directory (MUTTMENU *menu, struct browser_state *state,
   if ((dp = opendir (d)) == NULL)
   {
     mutt_perror (d);
-    return (-1);
+    return -1;
   }
 
   init_state (state, menu);
@@ -716,7 +716,7 @@ static int examine_vfolders (MUTTMENU *menu, struct browser_state *state)
   BUFFY *tmp = VirtIncoming;
 
   if (!VirtIncoming)
-    return (-1);
+    return -1;
   mutt_buffy_check (0);
 
   init_state (state, menu);
@@ -765,7 +765,7 @@ static int examine_mailboxes (MUTTMENU *menu, struct browser_state *state)
   BUFFY *tmp = Incoming;
 
   if (!Incoming)
-    return (-1);
+    return -1;
   mutt_buffy_check (0);
 
   init_state (state, menu);
index 9654ace303fa9b6e6f97d28a69416f4707e399b7..4d2594a3f37c9df34f1fb1f88d1b36ac4ed1adbe 100644 (file)
--- a/buffer.c
+++ b/buffer.c
@@ -249,14 +249,14 @@ int mutt_extract_token (BUFFER *dest, BUFFER *tok, int flags)
       if (!pc)
       {
        mutt_debug (1, "mutt_get_token: mismatched backticks\n");
-       return (-1);
+       return -1;
       }
       cmd = mutt_substrdup (tok->dptr, pc);
       if ((pid = mutt_create_filter (cmd, NULL, &fp, NULL)) < 0)
       {
        mutt_debug (1, "mutt_get_token: unable to fork command: %s\n", cmd);
        FREE (&cmd);
-       return (-1);
+       return -1;
       }
       FREE (&cmd);
 
diff --git a/buffy.c b/buffy.c
index 5dadf0435b0117dfe283e19a823b3af1d594e086..4da236330bde642f3787cfd9cd7e4b43bf6a7a53 100644 (file)
--- a/buffy.c
+++ b/buffy.c
@@ -93,10 +93,10 @@ static int fseek_last_message (FILE * f)
   if (mutt_strncmp ("From ", buffer, 5) == 0)
   {
     fseek (f, 0, 0);
-    return (0);
+    return 0;
   }
 
-  return (-1);
+  return -1;
 }
 
 /* Return 1 if the last message is new */
@@ -107,7 +107,7 @@ static int test_last_status_new (FILE * f)
   int result = 0;
 
   if (fseek_last_message (f) == -1)
-    return (0);
+    return 0;
 
   hdr = mutt_new_header ();
   tmp_envelope = mutt_read_rfc822_header (f, hdr, 0, 0);
@@ -766,7 +766,7 @@ int mutt_buffy_check (int force)
 #endif
 
   BuffyDoneTime = BuffyTime;
-  return (BuffyCount);
+  return BuffyCount;
 }
 
 int mutt_buffy_list (void)
@@ -814,13 +814,13 @@ int mutt_buffy_list (void)
   if (!first)
   {
     mutt_message ("%s", buffylist);
-    return (1);
+    return 1;
   }
   /* there were no mailboxes needing to be notified, so clean up since
    * BuffyNotify has somehow gotten out of sync
    */
   BuffyNotify = 0;
-  return (0);
+  return 0;
 }
 
 void mutt_buffy_setnotified (const char *path)
@@ -841,7 +841,7 @@ int mutt_buffy_notify (void)
   {
     return (mutt_buffy_list ());
   }
-  return (0);
+  return 0;
 }
 
 /*
diff --git a/color.c b/color.c
index 57c9c280806513285e9afc0cb1672cc75850a92c..abec4ee80649e59d39ace4fb8f15d5280374dca0 100644 (file)
--- a/color.c
+++ b/color.c
@@ -133,7 +133,7 @@ static COLOR_LINE *new_color_line (void)
 
   p->fg = p->bg = -1;
 
-  return (p);
+  return p;
 }
 
 static void free_color_line(COLOR_LINE **l,
@@ -248,7 +248,7 @@ int mutt_alloc_color (int fg, int bg)
   }
 
   /* check to see if there are colors left */
-  if (++UserColors > COLOR_PAIRS) return (A_NORMAL);
+  if (++UserColors > COLOR_PAIRS) return A_NORMAL;
 
   /* find the smallest available index (object) */
   i = 1;
@@ -357,13 +357,13 @@ parse_color_name (const char *s, int *col, int *attr, int is_fg, BUFFER *err)
        (*col >= COLORS && !option(OPTNOCURSES) && has_colors()))
     {
       snprintf (err->data, err->dsize, _("%s: color not supported by term"), s);
-      return (-1);
+      return -1;
     }
   }
   else if ((*col = mutt_getvaluebyname (s, Colors)) == -1)
   {
     snprintf (err->data, err->dsize, _("%s: no such color"), s);
-    return (-1);
+    return -1;
   }
 
   if (is_bright)
@@ -454,7 +454,7 @@ static int _mutt_parse_uncolor (BUFFER *buf, BUFFER *s, unsigned long data,
   if ((object = mutt_getvaluebyname (buf->data, Fields)) == -1)
   {
     snprintf (err->data, err->dsize, _("%s: no such object"), buf->data);
-    return (-1);
+    return -1;
   }
 
   if (object > MT_COLOR_INDEX_SUBJECT) { /* uncolor index column */
@@ -470,14 +470,14 @@ static int _mutt_parse_uncolor (BUFFER *buf, BUFFER *s, unsigned long data,
     snprintf (err->data, err->dsize,
              _("%s: command valid only for index, body, header objects"),
              parse_uncolor ? "uncolor" : "unmono");
-    return (-1);
+    return -1;
   }
 
   if (!MoreArgs (s))
   {
     snprintf (err->data, err->dsize,
              _("%s: too few arguments"), parse_uncolor ? "uncolor" : "unmono");
-    return (-1);
+    return -1;
   }
 
   if(
@@ -529,7 +529,7 @@ static int _mutt_parse_uncolor (BUFFER *buf, BUFFER *s, unsigned long data,
     for (i = 0; Context && i < Context->msgcount; i++)
       Context->hdrs[i]->pair = 0;
   }
-  return (0);
+  return 0;
 }
 
 #ifdef HAVE_COLOR
@@ -619,7 +619,7 @@ add_pattern (COLOR_LINE **top, const char *s, int sensitive,
     {
       regerror (r, &tmp->rx, err->data, err->dsize);
       free_color_line(&tmp, 1);
-      return (-1);
+      return -1;
     }
     tmp->next = *top;
     tmp->pattern = safe_strdup (s);
@@ -671,7 +671,7 @@ parse_object(BUFFER *buf, BUFFER *s, int *o, int *ql, BUFFER *err)
   else if ((*o = mutt_getvaluebyname (buf->data, Fields)) == -1)
   {
     snprintf (err->data, err->dsize, _("%s: no such object"), buf->data);
-    return (-1);
+    return -1;
   }
 
   return 0;
@@ -687,24 +687,24 @@ parse_color_pair(BUFFER *buf, BUFFER *s, int *fg, int *bg, int *attr, BUFFER *er
   if (! MoreArgs (s))
   {
     strfcpy (err->data, _("color: too few arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   mutt_extract_token (buf, s, 0);
 
   if (parse_color_name (buf->data, fg, attr, 1, err) != 0)
-    return (-1);
+    return -1;
 
   if (! MoreArgs (s))
   {
     strfcpy (err->data, _("color: too few arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   mutt_extract_token (buf, s, 0);
 
   if (parse_color_name (buf->data, bg, attr, 0, err) != 0)
-    return (-1);
+    return -1;
 
   return 0;
 }
@@ -721,7 +721,7 @@ parse_attr_spec(BUFFER *buf, BUFFER *s, int *fg, int *bg, int *attr, BUFFER *err
   if (! MoreArgs (s))
   {
     strfcpy (err->data, _("mono: too few arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   mutt_extract_token (buf, s, 0);
@@ -741,7 +741,7 @@ parse_attr_spec(BUFFER *buf, BUFFER *s, int *fg, int *bg, int *attr, BUFFER *err
   else
   {
     snprintf (err->data, err->dsize, _("%s: no such attribute"), buf->data);
-    return (-1);
+    return -1;
   }
 
   return 0;
@@ -797,7 +797,7 @@ _mutt_parse_color (BUFFER *buf, BUFFER *s, BUFFER *err,
   if (MoreArgs (s) && (object != MT_COLOR_STATUS))
   {
     strfcpy (err->data, _("too many arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   /* dry run? */
@@ -817,7 +817,7 @@ _mutt_parse_color (BUFFER *buf, BUFFER *s, BUFFER *err,
     && use_default_colors () != OK)
   {
     strfcpy (err->data, _("default colors not supported"), err->dsize);
-    return (-1);
+    return -1;
   }
 # endif /* HAVE_USE_DEFAULT_COLORS */
 #endif
@@ -915,7 +915,7 @@ _mutt_parse_color (BUFFER *buf, BUFFER *s, BUFFER *err,
       set_option (OPTFORCEREDRAWINDEX);
   }
 
-  return (r);
+  return r;
 }
 
 #ifdef HAVE_COLOR
index 193f48dd931c32960639a290237ea1c81b521909..e7d4339cdd8f6f2535a3ac0c6991a96d70940ccb 100644 (file)
@@ -116,7 +116,7 @@ int mutt_display_message (HEADER *cur)
   if ((fpout = safe_fopen (tempfile, "w")) == NULL)
   {
     mutt_error (_("Could not create temporary file!"));
-    return (0);
+    return 0;
   }
 
   if (DisplayFilter && *DisplayFilter)
@@ -795,7 +795,7 @@ int mutt_save_message (HEADER *h, int delete,
 
   mutt_pretty_mailbox (buf, sizeof (buf));
   if (mutt_enter_fname (prompt, buf, sizeof (buf), redraw, 0) == -1)
-    return (-1);
+    return -1;
 
   if (*redraw != REDRAW_FULL)
   {
@@ -806,7 +806,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>
@@ -921,7 +921,7 @@ int mutt_save_message (HEADER *h, int delete,
       mutt_buffy_cleanup (buf, &st);
 
     mutt_clear_error ();
-    return (0);
+    return 0;
   }
 
   return -1;
index 1b00326f6e73782318a881ce0dc2e22b17d1c6c0..db4e5af2ed316b5ab1083444c64937313a3be61d 100644 (file)
@@ -185,7 +185,7 @@ int mutt_complete (char *s, size_t slen)
   {
     mutt_debug (1, "mutt_complete(): %s: %s (errno %d).\n",
                 exp_dirpart, strerror (errno), errno);
-    return (-1);
+    return -1;
   }
 
   /*
index 37050099c8b9f69128c22dda4df15672ce5bf685..ab6120297fa623945b080ddac986ca0888412e3c 100644 (file)
--- a/compose.c
+++ b/compose.c
@@ -340,7 +340,7 @@ static int edit_address_list (int line, ADDRESS **addr)
   if (option (OPTNEEDREDRAW))
   {
     unset_option (OPTNEEDREDRAW);
-    return (REDRAW_FULL);
+    return REDRAW_FULL;
   }
 
   if (mutt_addrlist_to_intl (*addr, &err) != 0)
@@ -370,7 +370,7 @@ static int delete_attachment (MUTTMENU *menu, short *idxlen, int x)
   {
     mutt_error (_("You may not delete the only attachment."));
     idx[x]->content->tagged = 0;
-    return (-1);
+    return -1;
   }
 
   for (y = 0; y < *idxlen; y++)
@@ -392,7 +392,7 @@ static int delete_attachment (MUTTMENU *menu, short *idxlen, int x)
   idx[*idxlen - 1] = NULL;
   menu->max = --(*idxlen);
 
-  return (0);
+  return 0;
 }
 
 static void update_idx (MUTTMENU *menu, ATTACHPTR **idx, short idxlen)
@@ -501,7 +501,7 @@ compose_format_str (char *buf, size_t buflen, size_t col, int cols, char op, con
 
     case 0:
       *buf = 0;
-      return (src);
+      return src;
 
     default:
       snprintf (buf, buflen, "%%%s%c", prefix, op);
@@ -513,7 +513,7 @@ compose_format_str (char *buf, size_t buflen, size_t col, int cols, char op, con
   else if (flags & MUTT_FORMAT_OPTIONAL)
     compose_status_line (buf, buflen, col, cols, menu, elsestring);
 
-  return (src);
+  return src;
 }
 
 static void compose_status_line (char *buf, size_t buflen, size_t col, int cols,
@@ -1511,6 +1511,6 @@ int mutt_compose_menu (HEADER *msg,   /* structure for new message */
 
   FREE (&idx);
 
-  return (r);
+  return r;
 }
 
diff --git a/copy.c b/copy.c
index 767361c8c5dd59c347ea9a77cbcb632695b3535b..f449a19bfd50e26f647c387adcbc7a4f30ba37a6 100644 (file)
--- a/copy.c
+++ b/copy.c
@@ -126,7 +126,7 @@ mutt_copy_hdr (FILE *in, FILE *out, LOFF_T off_start, LOFF_T off_end, int flags,
       }
 
       if (!ignore && fputs (buf, out) == EOF)
-       return (-1);
+       return -1;
     }
     return 0;
   }
@@ -320,8 +320,8 @@ mutt_copy_hdr (FILE *in, FILE *out, LOFF_T off_start, LOFF_T off_end, int flags,
   FREE (&headers);
 
   if (error)
-    return (-1);
-  return (0);
+    return -1;
+  return 0;
 }
 
 /* flags
@@ -654,7 +654,7 @@ _mutt_copy_message (FILE *fpout, FILE *fpin, HEADER *hdr, BODY *body,
        hdr->content->type == TYPEMULTIPART)
     {
       if (crypt_pgp_decrypt_mime (fpin, &fp, hdr->content, &cur))
-       return (-1);
+       return -1;
       fputs ("MIME-Version: 1.0\n", fpout);
     }
 
@@ -663,7 +663,7 @@ _mutt_copy_message (FILE *fpout, FILE *fpin, HEADER *hdr, BODY *body,
             && hdr->content->type == TYPEAPPLICATION)
     {
       if (crypt_smime_decrypt_mime (fpin, &fp, hdr->content, &cur))
-       return (-1);
+       return -1;
     }
 
     if (!cur)
@@ -680,7 +680,7 @@ _mutt_copy_message (FILE *fpout, FILE *fpin, HEADER *hdr, BODY *body,
     {
       safe_fclose (&fp);
       mutt_free_body (&cur);
-      return (-1);
+      return -1;
     }
     mutt_free_body (&cur);
     safe_fclose (&fp);
index 31926a3d578f412ddb930a8ac1672231f39b205b..789d67e53aaed5733736fe5f8cd7ab6c041a4a24 100644 (file)
@@ -1854,7 +1854,7 @@ int pgp_gpgme_decrypt_mime (FILE *fpin, FILE **fpout, BODY *b, BODY **cur)
     if ((decoded_fp = safe_fopen (tempfile, "w+")) == NULL)
     {
       mutt_perror (tempfile);
-      return (-1);
+      return -1;
     }
     unlink (tempfile);
 
@@ -2958,7 +2958,7 @@ static const char *crypt_entry_fmt (char *dest,
     mutt_FormatString (dest, destlen, col, cols, ifstring, mutt_attach_fmt, data, 0);
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, mutt_attach_fmt, data, 0);
-  return (src);
+  return src;
 }
 
 /* Used by the display function to format a line. */
@@ -4535,7 +4535,7 @@ static char *find_keys (ADDRESS *adrlist, unsigned int app, int oppenc_mode)
 
       mutt_free_list (&crypt_hook_list);
     }
-  return (keylist);
+  return keylist;
 }
 
 char *pgp_gpgme_findkeys (ADDRESS *adrlist, int oppenc_mode)
@@ -4824,7 +4824,7 @@ static int gpgme_send_menu (HEADER *msg, int *redraw, int is_smime)
     }
   }
 
-  return (msg->security);
+  return msg->security;
 }
 
 int pgp_gpgme_send_menu (HEADER *msg, int *redraw)
diff --git a/crypt.c b/crypt.c
index e1c7d53a31d1937a915fff0f2a72b867a21808b5..a1705c18de831c8b383c47012fb396fd6f0c68a3 100644 (file)
--- a/crypt.c
+++ b/crypt.c
@@ -135,7 +135,7 @@ int mutt_protect (HEADER *msg, char *keylist)
     return 0;
 
   if ((msg->security & SIGN) && !crypt_valid_passphrase (msg->security))
-    return (-1);
+    return -1;
 
   if ((WithCrypto & APPLICATION_PGP) && ((msg->security & PGPINLINE) == PGPINLINE))
   {
@@ -242,7 +242,7 @@ int mutt_protect (HEADER *msg, char *keylist)
       if (!tmp_pbody)
       {
        /* signed ? free it! */
-       return (-1);
+       return -1;
       }
       /* free tmp_body if messages was signed AND encrypted ... */
       if (tmp_smime_pbody != msg->content && tmp_smime_pbody != tmp_pbody)
@@ -284,7 +284,7 @@ int mutt_protect (HEADER *msg, char *keylist)
          mutt_free_body (&tmp_pgp_pbody->next);
        }
 
-       return (-1);
+       return -1;
       }
 
       /* destroy temporary signature envelope when doing retainable
@@ -833,7 +833,7 @@ int crypt_get_keys (HEADER *msg, char **keylist, int oppenc_mode)
        if ((*keylist = crypt_pgp_findkeys (adrlist, oppenc_mode)) == NULL)
        {
            rfc822_free_address (&adrlist);
-           return (-1);
+           return -1;
        }
        unset_option (OPTPGPCHECKTRUST);
      }
@@ -843,14 +843,14 @@ int crypt_get_keys (HEADER *msg, char **keylist, int oppenc_mode)
        if ((*keylist = crypt_smime_findkeys (adrlist, oppenc_mode)) == NULL)
        {
            rfc822_free_address (&adrlist);
-           return (-1);
+           return -1;
        }
      }
   }
 
   rfc822_free_address (&adrlist);
 
-  return (0);
+  return 0;
 }
 
 
index 1ee00da72136d91ad365f9fba223170853f6f49b..87c5822cbda8dfd88878f68884f16f3d0e1cf265 100644 (file)
@@ -107,10 +107,10 @@ event_t mutt_getch (void)
   event_t timeout = {-2, OP_NULL};
 
   if (UngetCount)
-    return (UngetKeyEvents[--UngetCount]);
+    return UngetKeyEvents[--UngetCount];
 
   if (!option(OPTIGNOREMACROEVENTS) && MacroBufferCount)
-    return (MacroEvents[--MacroBufferCount]);
+    return MacroEvents[--MacroBufferCount];
 
   SigInt = 0;
 
@@ -177,7 +177,7 @@ int _mutt_get_field (const char *field, char *buf, size_t buflen, int complete,
   mutt_window_clearline (MuttMessageWindow, 0);
   mutt_free_enter_state (&es);
 
-  return (ret);
+  return ret;
 }
 
 int mutt_get_field_unbuffered (char *msg, char *buf, size_t buflen, int flags)
@@ -188,7 +188,7 @@ int mutt_get_field_unbuffered (char *msg, char *buf, size_t buflen, int flags)
   rc = mutt_get_field (msg, buf, buflen, flags);
   unset_option (OPTIGNOREMACROEVENTS);
 
-  return (rc);
+  return rc;
 }
 
 void mutt_clear_error (void)
@@ -318,7 +318,7 @@ int mutt_yesorno (const char *msg, int def)
      * mutt_message() so it isn't displayed when the screen is refreshed. */
     mutt_clear_error();
   }
-  return (def);
+  return def;
 }
 
 /* this function is called when the user presses the abort key */
@@ -770,7 +770,7 @@ int mutt_any_key_to_continue (const char *s)
   close (f);
   fputs ("\r\n", stdout);
   mutt_clear_error ();
-  return (ch);
+  return ch;
 }
 
 int mutt_do_pager (const char *banner,
@@ -820,7 +820,7 @@ int _mutt_enter_fname (const char *prompt, char *buf, size_t blen,
   if (ch.ch < 0)
   {
     mutt_window_clearline (MuttMessageWindow, 0);
-    return (-1);
+    return -1;
   }
   else if (ch.ch == '?')
   {
index 55dab538efc68c6b22d3ad1bc9085a9645c6515e..592c3a81d4baf7e1ba40e6c806af6fa97136add9 100644 (file)
@@ -195,8 +195,8 @@ static int ci_next_undeleted (int msgno)
 
   for (i=msgno+1; i < Context->vcount; i++)
     if (! Context->hdrs[Context->v2r[i]]->deleted)
-      return (i);
-  return (-1);
+      return i;
+  return -1;
 }
 
 static int ci_previous_undeleted (int msgno)
@@ -205,8 +205,8 @@ static int ci_previous_undeleted (int msgno)
 
   for (i=msgno-1; i>=0; i--)
     if (! Context->hdrs[Context->v2r[i]]->deleted)
-      return (i);
-  return (-1);
+      return i;
+  return -1;
 }
 
 /* Return the index of the first new message, or failing that, the first
@@ -224,13 +224,13 @@ static int ci_first_message (void)
          ! Context->hdrs[Context->v2r[i]]->deleted)
       {
        if (! Context->hdrs[Context->v2r[i]]->old)
-         return (i);
+         return i;
        else if (old == -1)
          old = i;
       }
     }
     if (old != -1)
-      return (old);
+      return old;
 
     /* If Sort is reverse and not threaded, the latest message is first.
      * If Sort is threaded, the latest message is first iff exactly one
@@ -3230,7 +3230,7 @@ int mutt_index_menu (void)
   }
 
   mutt_menu_destroy (&menu);
-  return (close);
+  return close;
 }
 
 void mutt_set_header_color (CONTEXT *ctx, HEADER *curhdr)
diff --git a/date.c b/date.c
index 1a903491bbef24efa6374185fd1cea5ee6f4673f..a66606d84df2e88f8f8e35fe41afb6192d046cfd 100644 (file)
--- a/date.c
+++ b/date.c
@@ -108,7 +108,7 @@ time_t mutt_mktime (struct tm *t, int local)
   if (local)
     g -= compute_tz (g, t);
 
-  return (g);
+  return g;
 }
 
 /* Return 1 if month is February of leap year, else 0 */
@@ -119,7 +119,7 @@ static int is_leap_year_feb (struct tm *tm)
     int y = tm->tm_year + 1900;
     return (((y & 3) == 0) && (((y % 100) != 0) || ((y % 400) == 0)));
   }
-  return (0);
+  return 0;
 }
 
 void mutt_normalize_time (struct tm *tm)
diff --git a/edit.c b/edit.c
index 239719b119b5871df3aa5098db17d43db14066ad..90fc6695772213d46214a72ba98efa83d9802aae 100644 (file)
--- a/edit.c
+++ b/edit.c
@@ -88,7 +88,7 @@ be_snarf_data (FILE *f, char **buf, int *bufmax, int *buflen, LOFF_T offset,
     safe_realloc (&buf, sizeof (char *) * (++*bufmax));
   }
   if (buf) buf[*buflen] = NULL;
-  return (buf);
+  return buf;
 }
 
 static char **
@@ -114,7 +114,7 @@ be_snarf_file (const char *path, char **buf, int *max, int *len, int verbose)
     snprintf(tmp, sizeof(tmp), "%s: %s\n", path, strerror(errno));
     addstr(tmp);
   }
-  return (buf);
+  return buf;
 }
 
 static int be_barf_file (const char *path, char **buf, int buflen)
@@ -126,12 +126,12 @@ static int be_barf_file (const char *path, char **buf, int buflen)
   {
     addstr (strerror (errno));
     addch ('\n');
-    return (-1);
+    return -1;
   }
   for (i = 0; i < buflen; i++) fputs (buf[i], f);
   if (fclose (f) == 0) return 0;
   printw ("fclose: %s\n", strerror (errno));
-  return (-1);
+  return -1;
 }
 
 static void be_free_memory (char **buf, int buflen)
@@ -187,7 +187,7 @@ be_include_messages (char *msg, char **buf, int *bufmax, int *buflen,
       printw (_("%d: invalid message number.\n"), n);
     msg = NULL;
   }
-  return (buf);
+  return buf;
 }
 
 static void be_print_header (ENVELOPE *env)
index e97a5616f078b3a8c23505b3bc85b782fb4a3900..4f743318f8f633b0d0dc88386c00c7aa3105d53f 100644 (file)
--- a/filter.c
+++ b/filter.c
@@ -39,7 +39,7 @@ mutt_create_filter_fd (const char *cmd, FILE **in, FILE **out, FILE **err,
   {
     *in = 0;
     if (pipe (pin) == -1)
-      return (-1);
+      return -1;
   }
 
   if (out)
@@ -52,7 +52,7 @@ mutt_create_filter_fd (const char *cmd, FILE **in, FILE **out, FILE **err,
        close (pin[0]);
        close (pin[1]);
       }
-      return (-1);
+      return -1;
     }
   }
 
@@ -71,7 +71,7 @@ mutt_create_filter_fd (const char *cmd, FILE **in, FILE **out, FILE **err,
        close (pout[0]);
        close (pout[1]);
       }
-      return (-1);
+      return -1;
     }
   }
 
@@ -148,7 +148,7 @@ mutt_create_filter_fd (const char *cmd, FILE **in, FILE **out, FILE **err,
       close (perr[1]);
     }
 
-    return (-1);
+    return -1;
   }
 
   if (out)
@@ -169,7 +169,7 @@ mutt_create_filter_fd (const char *cmd, FILE **in, FILE **out, FILE **err,
     *err = fdopen (perr[0], "r");
   }
 
-  return (thepid);
+  return thepid;
 }
 
 pid_t mutt_create_filter (const char *s, FILE **in, FILE **out, FILE **err)
diff --git a/flags.c b/flags.c
index d6e82c3c5d987d2d5769472d3db1a4330ab57427..709f7589d34b6f3b840c6f4731385fd706fd0844 100644 (file)
--- a/flags.c
+++ b/flags.c
@@ -300,7 +300,7 @@ int mutt_thread_set_flag (HEADER *hdr, int flag, int bf, int subthread)
   if ((Sort & SORT_MASK) != SORT_THREADS)
   {
     mutt_error (_("Threading is not enabled."));
-    return (-1);
+    return -1;
   }
 
   if (!subthread)
@@ -312,7 +312,7 @@ int mutt_thread_set_flag (HEADER *hdr, int flag, int bf, int subthread)
     mutt_set_flag (Context, cur->message, flag, bf);
 
   if ((cur = cur->child) == NULL)
-    return (0);
+    return 0;
 
   while (true)
   {
@@ -329,7 +329,7 @@ int mutt_thread_set_flag (HEADER *hdr, int flag, int bf, int subthread)
       {
        cur = cur->parent;
        if (cur == start)
-         return (0);
+         return 0;
       }
       cur = cur->next;
     }
@@ -351,7 +351,7 @@ int mutt_change_flag (HEADER *h, int bf)
   if (i < 0)
   {
     mutt_window_clearline (MuttMessageWindow, 0);
-    return (-1);
+    return -1;
   }
 
   mutt_window_clearline (MuttMessageWindow, 0);
@@ -399,7 +399,7 @@ int mutt_change_flag (HEADER *h, int bf)
 
     default:
       BEEP ();
-      return (-1);
+      return -1;
   }
 
   if (h)
diff --git a/from.c b/from.c
index 934f27bf5c7522c0c7ddbef6982f9f158988c070..be79ded5c0ac405520d7a12fae44e5acc1f5992b 100644 (file)
--- a/from.c
+++ b/from.c
@@ -36,8 +36,8 @@ int mutt_check_month (const char *s)
 
   for (i = 0; i < 12; i++)
     if (mutt_strncasecmp (s, Months[i], 3) == 0)
-      return (i);
-  return (-1); /* error */
+      return i;
+  return -1; /* error */
 }
 
 static int is_day_name (const char *s)
index 4919baa000840e11043c6086a5bdd817c7fc2d48..fdcfbc3344444529a781db55145f34fc4d54b0e8 100644 (file)
--- a/handler.c
+++ b/handler.c
@@ -929,7 +929,7 @@ static int is_mmnoask (const char *buf)
   if ((p = getenv ("MM_NOASK")) != NULL && *p)
   {
     if (mutt_strcmp (p, "1") == 0)
-      return (1);
+      return 1;
 
     strfcpy (tmp, p, sizeof (tmp));
     p = tmp;
@@ -941,26 +941,26 @@ static int is_mmnoask (const char *buf)
        if (*(q+1) == '*')
        {
          if (ascii_strncasecmp (buf, p, q-p) == 0)
-           return (1);
+           return 1;
        }
        else
        {
          if (ascii_strcasecmp (buf, p) == 0)
-           return (1);
+           return 1;
        }
       }
       else
       {
        lng = mutt_strlen (p);
        if (buf[lng] == '/' && (mutt_strncasecmp (buf, p, lng) == 0))
-         return (1);
+         return 1;
       }
 
       p = NULL;
     }
   }
 
-  return (0);
+  return 0;
 }
 
 /*
@@ -1235,9 +1235,9 @@ int mutt_can_decode (BODY *a)
   if (is_autoview (a))
     return 1;
   else if (a->type == TYPETEXT)
-    return (1);
+    return 1;
   else if (a->type == TYPEMESSAGE)
-    return (1);
+    return 1;
   else if (a->type == TYPEMULTIPART)
   {
     BODY *p = NULL;
@@ -1246,25 +1246,25 @@ int mutt_can_decode (BODY *a)
     {
       if ((ascii_strcasecmp (a->subtype, "signed") == 0) ||
          (ascii_strcasecmp (a->subtype, "encrypted") == 0))
-        return (1);
+        return 1;
     }
 
     for (p = a->parts; p; p = p->next)
     {
       if (mutt_can_decode (p))
-        return (1);
+        return 1;
     }
 
   }
   else if (WithCrypto && a->type == TYPEAPPLICATION)
   {
     if ((WithCrypto & APPLICATION_PGP) && mutt_is_application_pgp(a))
-      return (1);
+      return 1;
     if ((WithCrypto & APPLICATION_SMIME) && mutt_is_application_smime(a))
-      return (1);
+      return 1;
   }
 
-  return (0);
+  return 0;
 }
 
 static int multipart_handler (BODY *a, STATE *s)
diff --git a/hash.c b/hash.c
index 9b27136df45f440599aeec1af39fbfb829a9242b..2403367370dd69d3f078d7d7f0882b9378bd1184 100644 (file)
--- a/hash.c
+++ b/hash.c
@@ -170,7 +170,7 @@ static int union_hash_insert (HASH * table, union hash_key key, void *data)
       if (r == 0)
       {
        FREE (&ptr);
-       return (-1);
+       return -1;
       }
       if (r > 0)
        break;
@@ -212,7 +212,7 @@ static struct hash_elem *union_hash_find_elem (const HASH *table, union hash_key
   for (; ptr; ptr = ptr->next)
   {
     if (table->cmp_key (key, ptr->key) == 0)
-      return (ptr);
+      return ptr;
   }
   return NULL;
 }
index b3596ceb704d4ab6df076cd95ff1b4bd0f5893f0..449c332f6eaa8e8cbe3a07303b5e76786b307758 100644 (file)
--- a/hdrline.c
+++ b/hdrline.c
@@ -1231,7 +1231,7 @@ hdr_format_str (char *dest,
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, hdr_format_str, (unsigned long) hfi, flags);
 
-  return (src);
+  return src;
 #undef THREAD_NEW
 #undef THREAD_OLD
 }
diff --git a/help.c b/help.c
index 6cb3962a6e7ed02b618f0127dcb2bc31ba56ed3b..1f6242931fe9780b4eb41b94e16dbff1405aca08 100644 (file)
--- a/help.c
+++ b/help.c
@@ -48,7 +48,7 @@ static const struct binding_t *help_lookup_function (int op, int menu)
        return (&map[i]);
   }
 
-  return (NULL);
+  return NULL;
 }
 
 void mutt_make_help (char *d, size_t dlen, const char *txt, int menu, int op)
@@ -188,7 +188,7 @@ static int pad (FILE *f, int col, int i)
   {
     snprintf (fmt, sizeof(fmt), "%%-%ds", i - col);
     fprintf (f, fmt, "");
-    return (i);
+    return i;
   }
   fputc (' ', f);
   return (col + 1);
index f4a981811e9c83ba2ff5460eff2a7b2bf98f9a95..55b2eb05373375856bd8c2103394b9667523906a 100644 (file)
--- a/history.c
+++ b/history.c
@@ -251,7 +251,7 @@ char *mutt_history_next (history_class_t hclass)
   struct history *h = GET_HISTORY(hclass);
 
   if (!HistSize || !h)
-    return (""); /* disabled */
+    return ""; /* disabled */
 
   next = h->cur + 1;
   if (next > HistSize)
@@ -269,7 +269,7 @@ char *mutt_history_prev (history_class_t hclass)
   struct history *h = GET_HISTORY(hclass);
 
   if (!HistSize || !h)
-    return (""); /* disabled */
+    return ""; /* disabled */
 
   prev = h->cur - 1;
   if (prev < 0)
diff --git a/hook.c b/hook.c
index 2dca2cc2b7f20ec7ab452cf3f7d1f02aca1d35e0..e7ddd14ece819a57777878d3fd2b77ff1ad10e32 100644 (file)
--- a/hook.c
+++ b/hook.c
@@ -235,7 +235,7 @@ error:
   if (~data & MUTT_GLOBALHOOK) /* NOT a global hook */
     FREE (&pattern.data);
   FREE (&command.data);
-  return (-1);
+  return -1;
 }
 
 static void delete_hook (HOOK *h)
@@ -300,7 +300,7 @@ int mutt_parse_unhook (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
       {
        snprintf (err->data, err->dsize,
                 _("unhook: unknown hook type: %s"), buf->data);
-       return (-1);
+       return -1;
       }
       if (current_hook_type == type)
       {
@@ -362,9 +362,9 @@ char *mutt_find_hook (int type, const char *pat)
     if (tmp->type & type)
     {
       if (regexec (tmp->rx.rx, pat, 0, NULL, 0) == 0)
-       return (tmp->command);
+       return tmp->command;
     }
-  return (NULL);
+  return NULL;
 }
 
 void mutt_message_hook (CONTEXT *ctx, HEADER *hdr, int type)
@@ -492,9 +492,9 @@ static char *_mutt_string_hook (const char *match, int hook)
   {
     if ((tmp->type & hook) && ((match &&
         regexec (tmp->rx.rx, match, 0, NULL, 0) == 0) ^ tmp->rx.not))
-      return (tmp->command);
+      return tmp->command;
   }
-  return (NULL);
+  return NULL;
 }
 
 static LIST *_mutt_list_hook (const char *match, int hook)
@@ -508,7 +508,7 @@ static LIST *_mutt_list_hook (const char *match, int hook)
         ((match && regexec (tmp->rx.rx, match, 0, NULL, 0) == 0) ^ tmp->rx.not))
       matches = mutt_add_list (matches, tmp->command);
   }
-  return (matches);
+  return matches;
 }
 
 char *mutt_charset_hook (const char *chs)
index 87e2ae995e9859a499d104e1e23731301360de27..3de81d187c6a776d4d382e9e8c5bd937f86dd21e 100644 (file)
@@ -833,7 +833,7 @@ static int imap_open_new_message (MESSAGE *msg, CONTEXT *dest, HEADER *hdr)
   if ((msg->fp = safe_fopen (tmp, "w")) == NULL)
   {
     mutt_perror (tmp);
-    return (-1);
+    return -1;
   }
   msg->path = safe_strdup(tmp);
   return 0;
diff --git a/init.c b/init.c
index b0dec0d25c447b5e446eec09ecd728c9e62d3948..c62f8dad511aa615141e2af4dd40f85d1a82e9c2 100644 (file)
--- a/init.c
+++ b/init.c
@@ -60,7 +60,7 @@
        { \
          snprintf (err->data, err->dsize, \
            _("Not available in this menu.")); \
-         return (-1); \
+         return -1; \
        }
 
 typedef struct myvar
@@ -202,12 +202,12 @@ int query_quadoption (int opt, const char *prompt)
   {
     case MUTT_YES:
     case MUTT_NO:
-      return (v);
+      return v;
 
     default:
       v = mutt_yesorno (prompt, (v == MUTT_ASKYES));
       mutt_window_clearline (MuttMessageWindow, 0);
-      return (v);
+      return v;
   }
 
   /* not reached */
@@ -222,7 +222,7 @@ int mutt_option_index (const char *s)
   for (i = 0; MuttVars[i].option; i++)
     if (mutt_strcmp (s, MuttVars[i].option) == 0)
       return (MuttVars[i].type == DT_SYN ?  mutt_option_index ((char *) MuttVars[i].data) : i);
-  return (-1);
+  return -1;
 }
 
 #ifdef USE_LUA
@@ -332,7 +332,7 @@ parse_sort (short *val, const char *s, const struct mapping_t *map, BUFFER *err)
   if ((i = mutt_getvaluebyname (s, map)) == -1)
   {
     snprintf (err->data, err->dsize, _("%s: unknown sorting method"), s);
-    return (-1);
+    return -1;
   }
 
   *val = i | flags;
@@ -1712,7 +1712,7 @@ static int parse_alias (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   if (!MoreArgs (s))
   {
     strfcpy (err->data, _("alias: no address"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   mutt_extract_token (buf, s, 0);
@@ -1846,7 +1846,7 @@ static int parse_my_hdr (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err
   if ((p = strpbrk (buf->data, ": \t")) == NULL || *p != ':')
   {
     strfcpy (err->data, _("invalid header field"), err->dsize);
-    return (-1);
+    return -1;
   }
   keylen = p - buf->data + 1;
 
@@ -2291,7 +2291,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
        !(reset && (mutt_strcmp ("all", tmp->data) == 0)))
     {
       snprintf (err->data, err->dsize, _("%s: unknown variable"), tmp->data);
-      return (-1);
+      return -1;
     }
     SKIPWS (s->dptr);
 
@@ -2300,13 +2300,13 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
       if (query || unset || inv)
       {
        snprintf (err->data, err->dsize, _("prefix is illegal with reset"));
-       return (-1);
+       return -1;
       }
 
       if (s && *s->dptr == '=')
       {
        snprintf (err->data, err->dsize, _("value is illegal with reset"));
-       return (-1);
+       return -1;
       }
 
       if (mutt_strcmp ("all", tmp->data) == 0)
@@ -2314,7 +2314,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
        if (CurrentMenu == MENU_PAGER)
        {
          snprintf (err->data, err->dsize, _("Not available in this menu."));
-         return (-1);
+         return -1;
        }
        for (idx = 0; MuttVars[idx].option; idx++)
          restore_default (&MuttVars[idx]);
@@ -2342,7 +2342,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
        if (unset || inv || query)
        {
          snprintf (err->data, err->dsize, _("Usage: set variable=yes|no"));
-         return (-1);
+         return -1;
        }
 
        s->dptr++;
@@ -2354,7 +2354,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
        else
        {
          snprintf (err->data, err->dsize, _("Usage: set variable=yes|no"));
-         return (-1);
+         return -1;
        }
       }
 
@@ -2407,7 +2407,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
           else
           {
             snprintf (err->data, err->dsize, _("%s: unknown variable"), myvar);
-            return (-1);
+            return -1;
           }
         }
        else if (DTYPE (MuttVars[idx].type) == DT_ADDR)
@@ -2474,7 +2474,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
          {
            snprintf (err->data, err->dsize, _("Invalid value for option %s: \"%s\""),
                      MuttVars[idx].option, tmp->data);
-           return (-1);
+           return -1;
          }
 
          FREE ((void *) MuttVars[idx].data);           /* __FREE_CHECKED__ */
@@ -2798,7 +2798,7 @@ static int parse_set (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err)
 #endif
     }
   }
-  return (r);
+  return r;
 }
 
 /* Stack structure
@@ -2873,7 +2873,7 @@ static int source_rc (const char *rcfile_path, BUFFER *err)
       if (!to_absolute_path(rcfile, mutt_front_list(MuttrcStack)))
       {
         mutt_error("Error: impossible to build path of '%s'.", rcfile_path);
-        return (-1);
+        return -1;
       }
 
       if (!MuttrcStack || mutt_find_list(MuttrcStack, rcfile) == NULL)
@@ -2883,7 +2883,7 @@ static int source_rc (const char *rcfile_path, BUFFER *err)
       else
       {
         mutt_error("Error: Cyclic sourcing of configuration file '%s'.", rcfile);
-        return (-1);
+        return -1;
       }
   }
 
@@ -2892,7 +2892,7 @@ static int source_rc (const char *rcfile_path, BUFFER *err)
   if ((f = mutt_open_read (rcfile, &pid)) == NULL)
   {
     snprintf (err->data, err->dsize, "%s: %s", rcfile, strerror (errno));
-    return (-1);
+    return -1;
   }
 
   mutt_buffer_init (&token);
@@ -2940,7 +2940,7 @@ static int source_rc (const char *rcfile_path, BUFFER *err)
 
   mutt_pop_list(&MuttrcStack);
 
-  return (rc);
+  return rc;
 }
 
 #undef MAXERRS
@@ -2952,12 +2952,12 @@ static int parse_source (BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err
   if (mutt_extract_token (tmp, s, 0) != 0)
   {
     snprintf (err->data, err->dsize, _("source: error at %s"), s->dptr);
-    return (-1);
+    return -1;
   }
   if (MoreArgs (s))
   {
     strfcpy (err->data, _("source: too many arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
   strfcpy (path, tmp->data, sizeof (path));
   mutt_expand_path (path, sizeof (path));
@@ -3653,7 +3653,7 @@ const char *mutt_getnamebyvalue (int val, const struct mapping_t *map)
 
   for (i=0; map[i].name; i++)
     if (map[i].value == val)
-      return (map[i].name);
+      return map[i].name;
   return NULL;
 }
 
@@ -3663,8 +3663,8 @@ int mutt_getvaluebyname (const char *name, const struct mapping_t *map)
 
   for (i = 0; map[i].name; i++)
     if (ascii_strcasecmp (map[i].name, name) == 0)
-      return (map[i].value);
-  return (-1);
+      return map[i].value;
+  return -1;
 }
 
 #ifdef DEBUG
index 0efc1df9531b328e31429a93a513c1dad238bd4b..9b120bb117545959b7b0dd062b82110078eb83dc 100644 (file)
--- a/keymap.c
+++ b/keymap.c
@@ -133,7 +133,7 @@ static struct keymap_t *alloc_keys (int len, keycode_t *keys)
   p->len = len;
   p->keys = safe_malloc (len * sizeof (keycode_t));
   memcpy (p->keys, keys, len * sizeof (keycode_t));
-  return (p);
+  return p;
 }
 
 static int parse_fkey(char *s)
@@ -587,7 +587,7 @@ static const char *km_keyname (int c)
     snprintf (buf, sizeof (buf), "%c", (unsigned char) c);
   else
     snprintf (buf, sizeof (buf), "\\x%hx", (unsigned short) c);
-  return (buf);
+  return buf;
 }
 
 int km_expand_key (char *s, size_t len, struct keymap_t *map)
@@ -596,7 +596,7 @@ int km_expand_key (char *s, size_t len, struct keymap_t *map)
   int p = 0;
 
   if (!map)
-    return (0);
+    return 0;
 
   while (true)
   {
@@ -604,7 +604,7 @@ int km_expand_key (char *s, size_t len, struct keymap_t *map)
     len -= (l = mutt_strlen (s));
 
     if (++p >= map->len || !len)
-      return (1);
+      return 1;
 
     s += l;
   }
@@ -619,7 +619,7 @@ struct keymap_t *km_find_func (int menu, int func)
   for (; map; map = map->next)
     if (map->op == func)
       break;
-  return (map);
+  return map;
 }
 
 #ifdef NCURSES_VERSION
@@ -871,7 +871,7 @@ int mutt_parse_push (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   }
   else
     tokenize_push_macro_string (buf->data);
-  return (r);
+  return r;
 }
 
 /* expects to see: <menu-string>,<menu-string>,... <key-string> */
@@ -914,7 +914,7 @@ char *parse_keymap (int *menu, BUFFER *s, int maxmenus, int *nummenus, BUFFER *e
       strfcpy (err->data, _("null key sequence"), err->dsize);
     }
     else if (MoreArgs (s))
-      return (buf.data);
+      return buf.data;
   }
   else
   {
@@ -922,7 +922,7 @@ char *parse_keymap (int *menu, BUFFER *s, int maxmenus, int *nummenus, BUFFER *e
   }
 error:
   FREE (&buf.data);
-  return (NULL);
+  return NULL;
 }
 
 static int
@@ -934,9 +934,9 @@ try_bind (char *key, int menu, char *func, const struct binding_t *bindings)
     if (mutt_strcmp (func, bindings[i].name) == 0)
     {
       km_bindkey (key, menu, bindings[i].op);
-      return (0);
+      return 0;
     }
-  return (-1);
+  return -1;
 }
 
 const struct binding_t *km_get_table (int menu)
@@ -992,7 +992,7 @@ int mutt_parse_bind (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
 
   if ((key = parse_keymap (menu, s, sizeof (menu)/sizeof (menu[0]),
                           &nummenus, err)) == NULL)
-    return (-1);
+    return -1;
 
   /* function to execute */
   mutt_extract_token (buf, s, 0);
@@ -1028,7 +1028,7 @@ int mutt_parse_bind (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
     }
   }
   FREE (&key);
-  return (r);
+  return r;
 }
 
 /* macro <menu> <key> <macro> <description> */
@@ -1039,7 +1039,7 @@ int mutt_parse_macro (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   char *key = NULL;
 
   if ((key = parse_keymap (menu, s, sizeof (menu) / sizeof (menu[0]), &nummenus, err)) == NULL)
-    return (-1);
+    return -1;
 
   mutt_extract_token (buf, s, MUTT_TOKEN_CONDENSE);
   /* make sure the macro sequence is not an empty string */
@@ -1079,7 +1079,7 @@ int mutt_parse_macro (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
     }
   }
   FREE (&key);
-  return (r);
+  return r;
 }
 
 /* exec function-name */
@@ -1093,7 +1093,7 @@ int mutt_parse_exec (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   if (!MoreArgs (s))
   {
     strfcpy (err->data, _("exec: no arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   do
@@ -1113,7 +1113,7 @@ int mutt_parse_exec (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
     {
       mutt_flushinp ();
       mutt_error (_("%s: no such function"), function);
-      return (-1);
+      return -1;
     }
     nops++;
   }
diff --git a/lib.c b/lib.c
index 1757b3af2bf3527e65229c5c118dc2674428d29c..1cb5c88a06cec31c3eb26becf50b2bd8d7961173 100644 (file)
--- a/lib.c
+++ b/lib.c
@@ -145,7 +145,7 @@ void *safe_malloc (size_t siz)
     sleep (1);
     mutt_exit (1);
   }
-  return (p);
+  return p;
 }
 
 void safe_realloc (void *ptr, size_t siz)
@@ -232,7 +232,7 @@ char *safe_strdup (const char *s)
   l = strlen (s) + 1;
   p = safe_malloc (l);
   memcpy (p, s, l);
-  return (p);
+  return p;
 }
 
 char *safe_strcat (char *d, size_t l, const char *s)
@@ -297,7 +297,7 @@ char *mutt_strlower (char *s)
     p++;
   }
 
-  return (s);
+  return s;
 }
 
 /**
@@ -373,7 +373,7 @@ int mutt_copy_bytes (FILE *in, FILE *out, size_t size)
       break;
     if (fwrite (buf, 1, chunk, out) != chunk)
     {
-      return (-1);
+      return -1;
     }
     size -= chunk;
   }
@@ -390,7 +390,7 @@ int mutt_copy_stream (FILE *fin, FILE *fout)
   while ((l = fread (buf, 1, sizeof (buf), fin)) > 0)
   {
     if (fwrite (buf, 1, l, fout) != l)
-      return (-1);
+      return -1;
   }
 
   if (fflush(fout) != 0) return -1;
@@ -676,10 +676,10 @@ int safe_open (const char *path, int flags)
       compare_stat(&osb, &nsb) == -1)
   {
     close (fd);
-    return (-1);
+    return -1;
   }
 
-  return (fd);
+  return fd;
 }
 
 /* when opening files for writing, make sure the file doesn't already exist
@@ -702,7 +702,7 @@ FILE *safe_fopen (const char *path, const char *mode)
       flags |= O_WRONLY;
 
     if ((fd = safe_open (path, flags)) < 0)
-      return (NULL);
+      return NULL;
 
     return (fdopen (fd, mode));
   }
@@ -915,7 +915,7 @@ const char *mutt_stristr (const char *haystack, const char *needle)
   if (!haystack)
     return NULL;
   if (!needle)
-    return (haystack);
+    return haystack;
 
   while (*(p = haystack))
   {
@@ -925,7 +925,7 @@ const char *mutt_stristr (const char *haystack, const char *needle)
          p++, q++)
       ;
     if (!*q)
-      return (haystack);
+      return haystack;
     haystack++;
   }
   return NULL;
diff --git a/mbox.c b/mbox.c
index 1c91850ee6c341855640012da99d02504f094c49..069e0d5d2a2a405f63397a38dcf143d9bab249e6 100644 (file)
--- a/mbox.c
+++ b/mbox.c
@@ -62,7 +62,7 @@ static int mbox_lock_mailbox (CONTEXT *ctx, int excl, int retry)
     return 0;
   }
 
-  return (r);
+  return r;
 }
 
 static void mbox_unlock_mailbox (CONTEXT *ctx)
@@ -95,7 +95,7 @@ static int mmdf_parse_mailbox (CONTEXT *ctx)
   if (stat (ctx->path, &sb) == -1)
   {
     mutt_perror (ctx->path);
-    return (-1);
+    return -1;
   }
   ctx->atime = sb.st_atime;
   ctx->mtime = sb.st_mtime;
@@ -156,7 +156,7 @@ static int mmdf_parse_mailbox (CONTEXT *ctx)
        {
          mutt_debug (1, "mmdf_parse_mailbox: fseek() failed\n");
          mutt_error (_("Mailbox is corrupt!"));
-         return (-1);
+         return -1;
        }
       }
       else
@@ -213,7 +213,7 @@ static int mmdf_parse_mailbox (CONTEXT *ctx)
     {
       mutt_debug (1, "mmdf_parse_mailbox: corrupt mailbox!\n");
       mutt_error (_("Mailbox is corrupt!"));
-      return (-1);
+      return -1;
     }
   }
 
@@ -253,7 +253,7 @@ static int mbox_parse_mailbox (CONTEXT *ctx)
   if (stat (ctx->path, &sb) == -1)
   {
     mutt_perror (ctx->path);
-    return (-1);
+    return -1;
   }
 
   ctx->size = sb.st_size;
@@ -435,13 +435,13 @@ static int mbox_open_mailbox (CONTEXT *ctx)
   if ((ctx->fp = fopen (ctx->path, "r")) == NULL)
   {
     mutt_perror (ctx->path);
-    return (-1);
+    return -1;
   }
   mutt_block_signals ();
   if (mbox_lock_mailbox (ctx, 0, 1) == -1)
   {
     mutt_unblock_signals ();
-    return (-1);
+    return -1;
   }
 
   if (ctx->magic == MUTT_MBOX)
@@ -454,7 +454,7 @@ static int mbox_open_mailbox (CONTEXT *ctx)
 
   mbox_unlock_mailbox (ctx);
   mutt_unblock_signals ();
-  return (rc);
+  return rc;
 }
 
 static int mbox_open_mailbox_append (CONTEXT *ctx, int flags)
@@ -553,15 +553,15 @@ static int strict_addrcmp (const ADDRESS *a, const ADDRESS *b)
   {
     if ((mutt_strcmp (a->mailbox, b->mailbox) != 0) ||
        (mutt_strcmp (a->personal, b->personal) != 0))
-      return (0);
+      return 0;
 
     a = a->next;
     b = b->next;
   }
   if (a || b)
-    return (0);
+    return 0;
 
-  return (1);
+  return 1;
 }
 
 static int strict_cmp_lists (const LIST *a, const LIST *b)
@@ -569,15 +569,15 @@ static int strict_cmp_lists (const LIST *a, const LIST *b)
   while (a && b)
   {
     if (mutt_strcmp (a->data, b->data) != 0)
-      return (0);
+      return 0;
 
     a = a->next;
     b = b->next;
   }
   if (a || b)
-    return (0);
+    return 0;
 
-  return (1);
+  return 1;
 }
 
 static int strict_cmp_envelopes (const ENVELOPE *e1, const ENVELOPE *e2)
@@ -593,16 +593,16 @@ static int strict_cmp_envelopes (const ENVELOPE *e1, const ENVELOPE *e2)
        !strict_addrcmp (e1->to, e2->to) ||
        !strict_addrcmp (e1->cc, e2->cc) ||
        !strict_addrcmp (e1->return_path, e2->return_path))
-      return (0);
+      return 0;
     else
-      return (1);
+      return 1;
   }
   else
   {
     if (e1 == NULL && e2 == NULL)
-      return (1);
+      return 1;
     else
-      return (0);
+      return 0;
   }
 }
 
@@ -612,15 +612,15 @@ static int strict_cmp_parameters (const PARAMETER *p1, const PARAMETER *p2)
   {
     if ((mutt_strcmp (p1->attribute, p2->attribute) != 0) ||
        (mutt_strcmp (p1->value, p2->value) != 0))
-      return (0);
+      return 0;
 
     p1 = p1->next;
     p2 = p2->next;
   }
   if (p1 || p2)
-    return (0);
+    return 0;
 
-  return (1);
+  return 1;
 }
 
 static int strict_cmp_bodies (const BODY *b1, const BODY *b2)
@@ -631,8 +631,8 @@ static int strict_cmp_bodies (const BODY *b1, const BODY *b2)
       (mutt_strcmp (b1->description, b2->description) != 0) ||
       !strict_cmp_parameters (b1->parameter, b2->parameter) ||
       b1->length != b2->length)
-    return (0);
-  return (1);
+    return 0;
+  return 1;
 }
 
 /* return 1 if headers are strictly identical */
@@ -650,16 +650,16 @@ int mbox_strict_cmp_headers (const HEADER *h1, const HEADER *h2)
        h1->mime != h2->mime ||
        !strict_cmp_envelopes (h1->env, h2->env) ||
        !strict_cmp_bodies (h1->content, h2->content))
-      return (0);
+      return 0;
     else
-      return (1);
+      return 1;
   }
   else
   {
     if (h1 == NULL && h2 == NULL)
-      return (1);
+      return 1;
     else
-      return (0);
+      return 0;
   }
 }
 
@@ -754,7 +754,7 @@ static int reopen_mailbox (CONTEXT *ctx, int *index_hint)
     FREE (&old_hdrs);
 
     ctx->quiet = 0;
-    return (-1);
+    return -1;
   }
 
   mutt_touch_atime (fileno (ctx->fp));
@@ -860,13 +860,13 @@ static int mbox_check_mailbox (CONTEXT *ctx, int *index_hint)
   if (stat (ctx->path, &st) == 0)
   {
     if (st.st_mtime == ctx->mtime && st.st_size == ctx->size)
-      return (0);
+      return 0;
 
     if (st.st_size == ctx->size)
     {
       /* the file was touched, but it is still the same length, so just exit */
       ctx->mtime = st.st_mtime;
-      return (0);
+      return 0;
     }
 
     if (st.st_size > ctx->size)
@@ -882,7 +882,7 @@ static int mbox_check_mailbox (CONTEXT *ctx, int *index_hint)
           * probably the new mail arrived: no reason to wait till we can
           * parse it: we'll get it on the next pass
           */
-         return (MUTT_LOCKED);
+         return MUTT_LOCKED;
        }
        unlock = 1;
       }
@@ -918,7 +918,7 @@ static int mbox_check_mailbox (CONTEXT *ctx, int *index_hint)
            mutt_unblock_signals ();
          }
 
-         return (MUTT_NEW_MAIL); /* signal that new mail arrived */
+         return MUTT_NEW_MAIL; /* signal that new mail arrived */
        }
        else
          modified = 1;
@@ -942,7 +942,7 @@ static int mbox_check_mailbox (CONTEXT *ctx, int *index_hint)
        mbox_unlock_mailbox (ctx);
        mutt_unblock_signals ();
       }
-      return (MUTT_REOPENED);
+      return MUTT_REOPENED;
     }
   }
 
@@ -952,7 +952,7 @@ static int mbox_check_mailbox (CONTEXT *ctx, int *index_hint)
   mx_fastclose_mailbox (ctx);
   mutt_unblock_signals ();
   mutt_error (_("Mailbox was corrupted!"));
-  return (-1);
+  return -1;
 }
 
 /*
@@ -1034,7 +1034,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
   {
     mx_fastclose_mailbox (ctx);
     mutt_error (_("Fatal error!  Could not reopen mailbox!"));
-    return (-1);
+    return -1;
   }
 
   mutt_block_signals ();
@@ -1056,7 +1056,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
   }
   else if (i < 0)
     /* fatal error */
-    return (-1);
+    return -1;
 
   /* Create a temporary file to write the new version of the mailbox in. */
   mutt_mktemp (tempfile, sizeof (tempfile));
@@ -1220,7 +1220,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
     mutt_sleep (5);
     FREE (&newOffset);
     FREE (&oldOffset);
-    return (-1);
+    return -1;
   }
 
   if (fseeko (ctx->fp, offset, SEEK_SET) != 0 ||  /* seek the append location */
@@ -1285,7 +1285,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
     mutt_sleep (5);
     FREE (&newOffset);
     FREE (&oldOffset);
-    return (-1);
+    return -1;
   }
 
   /* Restore the previous access/modification times */
@@ -1300,7 +1300,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
     mutt_error (_("Fatal error!  Could not reopen mailbox!"));
     FREE (&newOffset);
     FREE (&oldOffset);
-    return (-1);
+    return -1;
   }
 
   /* update the offsets of the rewritten messages */
@@ -1326,7 +1326,7 @@ static int mbox_sync_mailbox (CONTEXT *ctx, int *index_hint)
       mutt_update_mailbox (tmp);
   }
 
-  return (0); /* signal success */
+  return 0; /* signal success */
 
 bail:  /* Come here in case of disaster */
 
@@ -1356,7 +1356,7 @@ bail:  /* Come here in case of disaster */
   {
     mutt_error (_("Could not reopen mailbox!"));
     mx_fastclose_mailbox (ctx);
-    return (-1);
+    return -1;
   }
 
   if (need_sort)
diff --git a/menu.c b/menu.c
index 2d7119649cf98558067a93de5c13dd60d8d67e85..3679d423beb9662d209142c2a28b5ae757294353 100644 (file)
--- a/menu.c
+++ b/menu.c
@@ -774,7 +774,7 @@ MUTTMENU *mutt_new_menu (int menu)
   p->messagewin = MuttMessageWindow;
   p->color = default_color;
   p->search = menu_search_generic;
-  return (p);
+  return p;
 }
 
 void mutt_menu_destroy (MUTTMENU **p)
@@ -811,7 +811,7 @@ static int menu_search (MUTTMENU *menu, int op)
     if (mutt_get_field ((op == OP_SEARCH || op == OP_SEARCH_NEXT)
                        ? _("Search for: ") : _("Reverse search for: "),
                        buf, sizeof (buf), MUTT_CLEAR) != 0 || !buf[0])
-      return (-1);
+      return -1;
     if (menu->menu >= 0 && menu->menu < MENU_MAX)
     {
       mutt_str_replace (&SearchBuffers[menu->menu], buf);
@@ -829,7 +829,7 @@ static int menu_search (MUTTMENU *menu, int op)
   {
     regerror (r, &re, buf, sizeof (buf));
     mutt_error ("%s", buf);
-    return (-1);
+    return -1;
   }
 
   r = menu->current + searchDir;
@@ -854,7 +854,7 @@ search_next:
   }
   regfree (&re);
   mutt_error (_("Not found."));
-  return (-1);
+  return -1;
 }
 
 static int menu_dialog_translate_op (int i)
@@ -908,7 +908,7 @@ int menu_redraw (MUTTMENU *menu)
   {
     menu_redraw_full (menu);
     /* allow the caller to do any local configuration */
-    return (OP_REDRAW);
+    return OP_REDRAW;
   }
 
   if (!menu->dialog)
@@ -1171,7 +1171,7 @@ int mutt_menu_loop (MUTTMENU *menu)
       default:
         if (menu->is_mailbox_list)
           last_position = menu->current;
-       return (i);
+       return i;
     }
   }
   /* not reached */
diff --git a/mh.c b/mh.c
index 7233507172004c34bd232190d3e26c35ae389466..c24a6e876a9f538fdfb7eeb78a435b437cdf91d0 100644 (file)
--- a/mh.c
+++ b/mh.c
@@ -382,7 +382,7 @@ static int mh_mkstemp (CONTEXT * dest, FILE ** fp, char **tgt)
     FREE (tgt);                /* __FREE_CHECKED__ */
     close (fd);
     unlink (path);
-    return (-1);
+    return -1;
   }
 
   return 0;
@@ -1305,7 +1305,7 @@ static int maildir_read_dir (CONTEXT * ctx)
    * of the main folder path from which to read messages
    */
   if (mh_read_dir (ctx, "new") == -1 || mh_read_dir (ctx, "cur") == -1)
-    return (-1);
+    return -1;
 
   return 0;
 }
@@ -1324,7 +1324,7 @@ static int maildir_open_mailbox_append (CONTEXT *ctx, int flags)
     if (mkdir (ctx->path, S_IRWXU))
     {
       mutt_perror (ctx->path);
-      return (-1);
+      return -1;
     }
 
     snprintf (tmp, sizeof (tmp), "%s/cur", ctx->path);
@@ -1332,7 +1332,7 @@ static int maildir_open_mailbox_append (CONTEXT *ctx, int flags)
     {
       mutt_perror (tmp);
       rmdir (ctx->path);
-      return (-1);
+      return -1;
     }
 
     snprintf (tmp, sizeof (tmp), "%s/new", ctx->path);
@@ -1342,7 +1342,7 @@ static int maildir_open_mailbox_append (CONTEXT *ctx, int flags)
       snprintf (tmp, sizeof (tmp), "%s/cur", ctx->path);
       rmdir (tmp);
       rmdir (ctx->path);
-      return (-1);
+      return -1;
     }
 
     snprintf (tmp, sizeof (tmp), "%s/tmp", ctx->path);
@@ -1354,7 +1354,7 @@ static int maildir_open_mailbox_append (CONTEXT *ctx, int flags)
       snprintf (tmp, sizeof (tmp), "%s/new", ctx->path);
       rmdir (tmp);
       rmdir (ctx->path);
-      return (-1);
+      return -1;
     }
   }
 
@@ -1376,7 +1376,7 @@ static int mh_open_mailbox_append (CONTEXT *ctx, int flags)
     if (mkdir (ctx->path, S_IRWXU))
     {
       mutt_perror (ctx->path);
-      return (-1);
+      return -1;
     }
 
     snprintf (tmp, sizeof (tmp), "%s/.mh_sequences", ctx->path);
@@ -1384,7 +1384,7 @@ static int mh_open_mailbox_append (CONTEXT *ctx, int flags)
     {
       mutt_perror (tmp);
       rmdir (ctx->path);
-      return (-1);
+      return -1;
     }
     close (i);
   }
@@ -1536,7 +1536,7 @@ static int maildir_open_new_message (MESSAGE * msg, CONTEXT * dest, HEADER * hdr
     FREE (&msg->path);
     close (fd);
     unlink (path);
-    return (-1);
+    return -1;
   }
 
   return 0;
@@ -1672,7 +1672,7 @@ static int _mh_commit_message (CONTEXT * ctx, MESSAGE * msg, HEADER * hdr,
   if ((dirp = opendir (ctx->path)) == NULL)
   {
     mutt_perror (ctx->path);
-    return (-1);
+    return -1;
   }
 
   /* figure out what the next message number is */
@@ -1834,7 +1834,7 @@ static int maildir_sync_message (CONTEXT * ctx, int msgno)
   {
     /* when doing attachment deletion/rethreading, fall back to the MH case. */
     if (mh_rewrite_message (ctx, msgno) != 0)
-      return (-1);
+      return -1;
   }
   else
   {
@@ -1851,7 +1851,7 @@ static int maildir_sync_message (CONTEXT * ctx, int msgno)
     {
       mutt_debug (1, "maildir_sync_message: %s: unable to find subdir!\n",
                   h->path);
-      return (-1);
+      return -1;
     }
     p++;
     strfcpy (newpath, p, sizeof (newpath));
@@ -1879,11 +1879,11 @@ static int maildir_sync_message (CONTEXT * ctx, int msgno)
     if (rename (oldpath, fullpath) != 0)
     {
       mutt_perror ("rename");
-      return (-1);
+      return -1;
     }
     mutt_str_replace (&h->path, partpath);
   }
-  return (0);
+  return 0;
 }
 
 #ifdef USE_HCACHE
index d2c10eab7c6cb9ad03d81e004ae03d0e8fb6a4b6..94d48b6f5a766c8ed0dfe1b321274c54b0cbcc64 100644 (file)
@@ -405,12 +405,12 @@ int mutt_lua_source_file(BUFFER *tmp, BUFFER *s, unsigned long data, BUFFER *err
   if (mutt_extract_token(tmp, s, 0) != 0)
   {
     snprintf(err->data, err->dsize, _("source: error at %s"), s->dptr);
-    return (-1);
+    return -1;
   }
   if (MoreArgs(s))
   {
     strfcpy(err->data, _("source: too many arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
   strfcpy(path, tmp->data, sizeof(path));
   mutt_expand_path(path, sizeof(path));
index 2072b2da4a1fcff7c61434b3d9511b3c352c0dbd..2b31b499c1f4299a02d2fd4c9380882268832e35 100644 (file)
@@ -503,7 +503,7 @@ static char *tls_make_date (time_t t, char *s, size_t len)
   else
     strfcpy (s, _("[invalid date]"), len);
 
-  return (s);
+  return s;
 }
 
 /*
index 7131530010896c1a290a5d42ac7bad20b569edb8..70928d5ed0ad6ec221c27a90528fedff26e15ca6 100644 (file)
--- a/muttlib.c
+++ b/muttlib.c
@@ -70,7 +70,7 @@ BODY *mutt_new_body (void)
 
   p->disposition = DISPATTACH;
   p->use_disp = 1;
-  return (p);
+  return p;
 }
 
 
@@ -341,7 +341,7 @@ int mutt_remove_from_rx_list (RX_LIST **l, const char *str)
       }
     }
   }
-  return (rv);
+  return rv;
 }
 
 void mutt_free_list (LIST **list)
@@ -375,7 +375,7 @@ LIST *mutt_copy_list (LIST *p)
     else
       l = r = t;
   }
-  return (l);
+  return l;
 }
 
 void mutt_free_header (HEADER **h)
@@ -590,7 +590,7 @@ char *_mutt_expand_path (char *s, size_t slen, int rx)
     imap_expand_path (s, slen);
 #endif
 
-  return (s);
+  return s;
 }
 
 /* Extract the real name from /etc/passwd's GECOS field.
@@ -644,7 +644,7 @@ char *mutt_get_parameter (const char *s, PARAMETER *p)
 {
   for (; p; p = p->next)
     if (ascii_strcasecmp (s, p->attribute) == 0)
-      return (p->value);
+      return p->value;
 
   return NULL;
 }
@@ -1121,7 +1121,7 @@ int mutt_check_overwrite (const char *attname, const char *path,
     strfcpy (tmp, mutt_basename (NONULL (attname)), sizeof (tmp));
     if (mutt_get_field (_("File under directory: "), tmp, sizeof (tmp),
                                     MUTT_FILE | MUTT_CLEAR) != 0 || !tmp[0])
-      return (-1);
+      return -1;
     mutt_concat_path (fname, path, tmp, flen);
   }
 
@@ -1758,16 +1758,16 @@ FILE *mutt_open_read (const char *path, pid_t *thepid)
   else
   {
     if (stat (path, &s) < 0)
-      return (NULL);
+      return NULL;
     if (S_ISDIR (s.st_mode))
     {
       errno = EINVAL;
-      return (NULL);
+      return NULL;
     }
     f = fopen (path, "r");
     *thepid = -1;
   }
-  return (f);
+  return f;
 }
 
 /* returns 0 if OK to proceed, -1 to abort, 1 to retry */
@@ -1840,7 +1840,7 @@ int mutt_save_confirm (const char *s, struct stat *st)
   }
 
   mutt_window_clearline (MuttMessageWindow, 0);
-  return (ret);
+  return ret;
 }
 
 void state_prefix_putc (char c, STATE *s)
diff --git a/mx.c b/mx.c
index d51fbcf51fbe17979461494a69fefab66ac5422d..59d5af64064014c702eb518470dff243f6654c99 100644 (file)
--- a/mx.c
+++ b/mx.c
@@ -456,7 +456,7 @@ int mx_get_magic (const char *path)
   {
     mutt_debug (1, "mx_get_magic(): unable to stat %s: %s (errno %d).\n",
                 path, strerror (errno), errno);
-    return (-1);
+    return -1;
   }
 
   if (S_ISDIR (st.st_mode))
@@ -473,9 +473,9 @@ int mx_get_magic (const char *path)
   {
     /* hard to tell what zero-length files are, so assume the default magic */
     if (DefaultMagic == MUTT_MBOX || DefaultMagic == MUTT_MMDF)
-      return (DefaultMagic);
+      return DefaultMagic;
     else
-      return (MUTT_MBOX);
+      return MUTT_MBOX;
   }
   else if ((f = fopen (path, "r")) != NULL)
   {
@@ -518,7 +518,7 @@ int mx_get_magic (const char *path)
   {
     mutt_debug (1, "mx_get_magic(): unable to open file %s for reading.\n",
                 path);
-    return (-1);
+    return -1;
   }
 
 #ifdef USE_COMPRESSED
@@ -527,7 +527,7 @@ int mx_get_magic (const char *path)
   if ((magic == 0) && mutt_comp_can_read (path))
     return MUTT_COMPRESSED;
 #endif
-  return (magic);
+  return magic;
 }
 
 /*
@@ -544,7 +544,7 @@ int mx_set_magic (const char *s)
   else if (ascii_strcasecmp (s, "maildir") == 0)
     DefaultMagic = MUTT_MAILDIR;
   else
-    return (-1);
+    return -1;
 
   return 0;
 }
@@ -669,7 +669,7 @@ CONTEXT *mx_open_mailbox (const char *path, int flags, CONTEXT *pctx)
     mx_fastclose_mailbox (ctx);
     if (!pctx)
       FREE (&ctx);
-    return (NULL);
+    return NULL;
   }
 
   mutt_make_label_hash (ctx);
@@ -709,7 +709,7 @@ CONTEXT *mx_open_mailbox (const char *path, int flags, CONTEXT *pctx)
   }
 
   unset_option (OPTFORCEREFRESH);
-  return (ctx);
+  return ctx;
 }
 
 /* free up memory associated with the mailbox context */
@@ -908,7 +908,7 @@ int mx_close_mailbox (CONTEXT *ctx, int *index_hint)
       if ((move_messages = query_quadoption (OPT_MOVE, buf)) == MUTT_ABORT)
       {
        ctx->closing = 0;
-       return (-1);
+       return -1;
       }
     }
   }
@@ -925,7 +925,7 @@ int mx_close_mailbox (CONTEXT *ctx, int *index_hint)
     if ((purge = query_quadoption (OPT_DELETE, buf)) == MUTT_ABORT)
     {
       ctx->closing = 0;
-      return (-1);
+      return -1;
     }
   }
 
@@ -1202,7 +1202,7 @@ int mx_sync_mailbox (CONTEXT *ctx, int *index_hint)
   {
     if (!ctx->quiet)
       mutt_message (_("Mailbox is unchanged."));
-    return (0);
+    return 0;
   }
 
   if (ctx->deleted)
@@ -1213,7 +1213,7 @@ int mx_sync_mailbox (CONTEXT *ctx, int *index_hint)
             ? _("Purge %d deleted message?") : _("Purge %d deleted messages?"),
              ctx->deleted);
     if ((purge = query_quadoption (OPT_DELETE, buf)) == MUTT_ABORT)
-      return (-1);
+      return -1;
     else if (purge == MUTT_NO)
     {
       if (!ctx->changed)
@@ -1295,7 +1295,7 @@ int mx_sync_mailbox (CONTEXT *ctx, int *index_hint)
     }
   }
 
-  return (rc);
+  return rc;
 }
 
 /* args:
@@ -1423,7 +1423,7 @@ int mx_close_message (CONTEXT *ctx, MESSAGE **msg)
 
   FREE (&(*msg)->commited_path);
   FREE (msg);          /* __FREE_CHECKED__ */
-  return (r);
+  return r;
 }
 
 void mx_alloc_memory (CONTEXT *ctx)
index d40398cf35f7ac76e9e50778283e00ce23788b8e..8bd4da3116bee0b347611829bfe8c30691316be9 100644 (file)
--- a/newsrc.c
+++ b/newsrc.c
@@ -879,7 +879,7 @@ nntp_format_str (char *dest, size_t destlen, size_t col, int cols, char op, cons
       snprintf (dest, destlen, tmp, acct->user);
       break;
   }
-  return (src);
+  return src;
 }
 
 /* Automatically loads a newsrc into memory, if necessary.
diff --git a/pager.c b/pager.c
index 278dbe0264602b9d9d331c5555798589e84efb2c..9330d0ca308b5e6d247df507dd7ddccc986adf1e 100644 (file)
--- a/pager.c
+++ b/pager.c
@@ -154,7 +154,7 @@ static int check_sig (const char *s, struct line_t *info, int n)
   }
 
   if (count == 0)
-    return (-1);
+    return -1;
 
   if (count > NumSigLines)
   {
@@ -166,10 +166,10 @@ static int check_sig (const char *s, struct line_t *info, int n)
       s++;
     }
 
-    return (-1);
+    return -1;
   }
 
-  return (0);
+  return 0;
 }
 
 static void
@@ -408,7 +408,7 @@ classify_quote (struct q_class_t **QuoteList, const char *qptr,
       class->color = ColorQuote[0];
       *QuoteList = class;
     }
-    return (*QuoteList);
+    return *QuoteList;
   }
 
   /* Did I mention how much I like emulating Lisp in C? */
@@ -1102,7 +1102,7 @@ fill_buffer (FILE *f, LOFF_T *last_pos, LOFF_T offset, unsigned char **buf,
     if ((*buf = (unsigned char *) mutt_read_line ((char *) *buf, blen, f, &l, MUTT_EOL)) == NULL)
     {
       fmt[0] = 0;
-      return (-1);
+      return -1;
     }
     *last_pos = ftello (f);
     b_read = (int) (*last_pos - offset);
@@ -1694,14 +1694,14 @@ mutt_pager (const char *banner, const char *fname, int flags, pager_t *extra)
   if ((fp = fopen (fname, "r")) == NULL)
   {
     mutt_perror (fname);
-    return (-1);
+    return -1;
   }
 
   if (stat (fname, &sb) != 0)
   {
     mutt_perror (fname);
     safe_fclose (&fp);
-    return (-1);
+    return -1;
   }
   unlink (fname);
 
diff --git a/parse.c b/parse.c
index 81e3f60c3fe9f3c5bc2029eab34a28ed3f5e7d28..460d9ee5e312f13747be3893aa6e599996a26cb4 100644 (file)
--- a/parse.c
+++ b/parse.c
@@ -49,12 +49,12 @@ char *mutt_read_rfc822_line (FILE *f, char *line, size_t *linelen)
        (ISSPACE (*line) && !offset))                   /* end of headers */
     {
       *line = 0;
-      return (line);
+      return line;
     }
 
     len = mutt_strlen (buf);
     if (! len)
-      return (line);
+      return line;
 
     buf += len - 1;
     if (*buf == '\n')
@@ -68,7 +68,7 @@ char *mutt_read_rfc822_line (FILE *f, char *line, size_t *linelen)
       if ((ch = fgetc (f)) != ' ' && ch != '\t')
       {
        ungetc (ch, f);
-       return (line); /* next line is a separate header field or EOH */
+       return line; /* next line is a separate header field or EOH */
       }
 
       /* eat tabs and spaces from the beginning of the continuation line */
@@ -115,23 +115,23 @@ static LIST *parse_references (char *s, int in_reply_to)
 int mutt_check_encoding (const char *c)
 {
   if (ascii_strncasecmp ("7bit", c, sizeof ("7bit")-1) == 0)
-    return (ENC7BIT);
+    return ENC7BIT;
   else if (ascii_strncasecmp ("8bit", c, sizeof ("8bit")-1) == 0)
-    return (ENC8BIT);
+    return ENC8BIT;
   else if (ascii_strncasecmp ("binary", c, sizeof ("binary")-1) == 0)
-    return (ENCBINARY);
+    return ENCBINARY;
   else if (ascii_strncasecmp ("quoted-printable", c, sizeof ("quoted-printable")-1) == 0)
-    return (ENCQUOTEDPRINTABLE);
+    return ENCQUOTEDPRINTABLE;
   else if (ascii_strncasecmp ("base64", c, sizeof("base64")-1) == 0)
-    return (ENCBASE64);
+    return ENCBASE64;
   else if (ascii_strncasecmp ("x-uuencode", c, sizeof("x-uuencode")-1) == 0)
-    return (ENCUUENCODED);
+    return ENCUUENCODED;
 #ifdef SUN_ATTACHMENT
   else if (ascii_strncasecmp ("uuencode", c, sizeof("uuencode")-1) == 0)
-    return (ENCUUENCODED);
+    return ENCUUENCODED;
 #endif
   else
-    return (ENCOTHER);
+    return ENCOTHER;
 }
 
 static PARAMETER *parse_parameters (const char *s)
@@ -255,7 +255,7 @@ static PARAMETER *parse_parameters (const char *s)
 bail:
 
   rfc2231_decode_parameters (&head);
-  return (head);
+  return head;
 }
 
 int mutt_check_mime_type (const char *s)
@@ -476,7 +476,7 @@ BODY *mutt_read_mime_header (FILE *fp, int digest)
 
   FREE (&line);
 
-  return (p);
+  return p;
 }
 
 void mutt_parse_part (FILE *fp, BODY *b)
@@ -553,7 +553,7 @@ BODY *mutt_parse_message_rfc822 (FILE *fp, BODY *parent)
     msg->length = 0;
 
   mutt_parse_part(fp, msg);
-  return (msg);
+  return msg;
 }
 
 /* parse a multipart structure
@@ -582,7 +582,7 @@ BODY *mutt_parse_multipart (FILE *fp, const char *boundary, LOFF_T end_off, int
   if (!boundary)
   {
     mutt_error (_("multipart message has no boundary parameter!"));
-    return (NULL);
+    return NULL;
   }
 
   blen = mutt_strlen (boundary);
@@ -657,7 +657,7 @@ BODY *mutt_parse_multipart (FILE *fp, const char *boundary, LOFF_T end_off, int
   for(last = head; last; last = last->next)
     mutt_parse_part(fp, last);
 
-  return (head);
+  return head;
 }
 
 static const char *uncomment_timezone (char *buf, size_t buflen, const char *tz)
@@ -780,20 +780,20 @@ time_t mutt_parse_date (const char *s, HEADER *h)
     {
       case 0: /* day of the month */
        if (mutt_atoi (t, &tm.tm_mday) < 0 || tm.tm_mday < 0)
-         return (-1);
+         return -1;
        if (tm.tm_mday > 31)
-         return (-1);
+         return -1;
        break;
 
       case 1: /* month of the year */
        if ((i = mutt_check_month (t)) < 0)
-         return (-1);
+         return -1;
        tm.tm_mon = i;
        break;
 
       case 2: /* year */
        if (mutt_atoi (t, &tm.tm_year) < 0 || tm.tm_year < 0)
-         return (-1);
+         return -1;
         if (tm.tm_year < 50)
          tm.tm_year += 100;
         else if (tm.tm_year >= 1900)
@@ -808,7 +808,7 @@ time_t mutt_parse_date (const char *s, HEADER *h)
        else
        {
          mutt_debug (1, "parse_date: could not process time format: %s\n", t);
-         return(-1);
+         return -1;
        }
        tm.tm_hour = hour;
        tm.tm_min = min;
@@ -873,7 +873,7 @@ time_t mutt_parse_date (const char *s, HEADER *h)
   if (count < 4) /* don't check for missing timezone */
   {
     mutt_debug (1, "parse_date(): error parsing date format, using received time\n");
-    return (-1);
+    return -1;
   }
 
   if (h)
@@ -1515,7 +1515,7 @@ ENVELOPE *mutt_read_rfc822_header (FILE *f, HEADER *hdr, short user_hdrs,
     }
   }
 
-  return (e);
+  return e;
 }
 
 ADDRESS *mutt_parse_adrlist (ADDRESS *p, const char *s)
index 35ff5e5dee47c12b1b17df974716f1c371391109..751aeb35e1eaf6156d5fd64ab880ce18910b627d 100644 (file)
--- a/pattern.c
+++ b/pattern.c
@@ -65,12 +65,12 @@ static int eat_regexp (pattern_t *pat, BUFFER *s, BUFFER *err)
       !buf.data)
   {
     snprintf (err->data, err->dsize, _("Error in expression: %s"), pexpr);
-    return (-1);
+    return -1;
   }
   if (!*buf.data)
   {
     snprintf (err->data, err->dsize, _("Empty expression"));
-    return (-1);
+    return -1;
   }
 
   if (pat->stringmatch)
@@ -94,7 +94,7 @@ static int eat_regexp (pattern_t *pat, BUFFER *s, BUFFER *err)
       mutt_buffer_printf (err, "'%s': %s", buf.data, errmsg);
       FREE (&buf.data);
       FREE (&pat->p.rx);
-      return (-1);
+      return -1;
     }
     FREE (&buf.data);
   }
@@ -375,12 +375,12 @@ static int eat_date (pattern_t *pat, BUFFER *s, BUFFER *err)
       || !buffer.data)
   {
     snprintf (err->data, err->dsize, _("Error in expression: %s"), pexpr);
-    return (-1);
+    return -1;
   }
   if (!*buffer.data)
   {
     snprintf (err->data, err->dsize, _("Empty expression"));
-    return (-1);
+    return -1;
   }
 
   memset (&min, 0, sizeof (min));
@@ -450,7 +450,7 @@ static int eat_date (pattern_t *pat, BUFFER *s, BUFFER *err)
       if ((pc = get_date (pc, &min, err)) == NULL)
       {
        FREE (&buffer.data);
-       return (-1);
+       return -1;
       }
       haveMin = true;
       SKIPWS (pc);
@@ -485,7 +485,7 @@ static int eat_date (pattern_t *pat, BUFFER *s, BUFFER *err)
       if (!parse_date_range (pc, &min, &max, haveMin, &baseMin, err))
       { /* bail out on any parsing error */
        FREE (&buffer.data);
-       return (-1);
+       return -1;
       }
     }
   }
@@ -928,7 +928,7 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
       if ((s.fpout = safe_fopen (tempfile, "w+")) == NULL)
       {
        mutt_perror (tempfile);
-       return (0);
+       return 0;
       }
 #endif
 
@@ -952,7 +952,7 @@ msg_search (CONTEXT *ctx, pattern_t* pat, int msgno)
            unlink (tempfile);
 #endif
          }
-         return (0);
+         return 0;
        }
 
        fseeko (msg->fp, h->offset, 0);
@@ -1311,7 +1311,7 @@ pattern_t *mutt_pattern_comp (/* const */ char *s, int flags, BUFFER *err)
     tmp->child = curlist;
     curlist = tmp;
   }
-  return (curlist);
+  return curlist;
 }
 
 static int
@@ -1469,7 +1469,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags, CONTEXT *ctx,
     case MUTT_THREAD:
       return (pat->not ^ match_threadcomplete(pat->child, flags, ctx, h->thread, 1, 1, 1, 1));
     case MUTT_ALL:
-      return (!pat->not);
+      return !pat->not;
     case MUTT_EXPIRED:
       return (pat->not ^ h->expired);
     case MUTT_SUPERSEDED:
@@ -1509,7 +1509,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags, CONTEXT *ctx,
 #ifdef USE_IMAP
       /* IMAP search sets h->matched at search compile time */
       if (ctx->magic == MUTT_IMAP && pat->stringmatch)
-       return (h->matched);
+       return h->matched;
 #endif
       return (pat->not ^ msg_search (ctx, pat, h->msgno));
     case MUTT_SENDER:
@@ -1638,7 +1638,7 @@ mutt_pattern_exec (struct pattern_t *pat, pattern_exec_flag flags, CONTEXT *ctx,
 #endif
   }
   mutt_error (_("error: unknown op %d (report this error)."), pat->op);
-  return (-1);
+  return -1;
 }
 
 static void quote_simple(char *tmp, size_t len, const char *p)
@@ -1789,7 +1789,7 @@ int mutt_pattern_func (int op, char *prompt)
   strfcpy (buf, NONULL (Context->pattern), sizeof (buf));
   if (prompt || op != MUTT_LIMIT)
   if (mutt_get_field (prompt, buf, sizeof (buf), MUTT_PATTERN | MUTT_CLEAR) != 0 || !buf[0])
-    return (-1);
+    return -1;
 
   mutt_message (_("Compiling search pattern..."));
 
@@ -1804,7 +1804,7 @@ int mutt_pattern_func (int op, char *prompt)
     FREE (&simple);
     mutt_error ("%s", err.data);
     FREE (&err.data);
-    return (-1);
+    return -1;
   }
 
 #ifdef USE_IMAP
@@ -1915,7 +1915,7 @@ int mutt_search_command (int cur, int op)
                        _("Search for: ") : _("Reverse search for: "),
                        buf, sizeof (buf),
                      MUTT_CLEAR | MUTT_PATTERN) != 0 || !buf[0])
-      return (-1);
+      return -1;
 
     if (op == OP_SEARCH || op == OP_SEARCH_NEXT)
       unset_option (OPTSEARCHREVERSE);
@@ -1942,7 +1942,7 @@ int mutt_search_command (int cur, int op)
        mutt_error ("%s", err.data);
        FREE (&err.data);
        LastSearch[0] = '\0';
-       return (-1);
+       return -1;
       }
       mutt_clear_error ();
     }
@@ -1977,7 +1977,7 @@ int mutt_search_command (int cur, int op)
       else
       {
         mutt_message (_("Search hit bottom without finding match"));
-       return (-1);
+       return -1;
       }
     }
     else if (i < 0)
@@ -1988,7 +1988,7 @@ int mutt_search_command (int cur, int op)
       else
       {
         mutt_message (_("Search hit top without finding match"));
-       return (-1);
+       return -1;
       }
     }
 
@@ -2021,12 +2021,12 @@ int mutt_search_command (int cur, int op)
     {
       mutt_error (_("Search interrupted."));
       SigInt = 0;
-      return (-1);
+      return -1;
     }
 
     i += incr;
   }
 
   mutt_error (_("Not found."));
-  return (-1);
+  return -1;
 }
diff --git a/pgp.c b/pgp.c
index 9c86519168dd383384cac2bfd39900b3b26af0ce..8399174496988be26aa9c6ace9cff65071ea6a22 100644 (file)
--- a/pgp.c
+++ b/pgp.c
@@ -87,7 +87,7 @@ int pgp_valid_passphrase (void)
   if (mutt_get_password (_("Enter PGP passphrase:"), PgpPass, sizeof (PgpPass)) == 0)
     {
       PgpExptime = time (NULL) + PgpTimeout;
-      return (1);
+      return 1;
     }
   else
     PgpExptime = 0;
@@ -845,7 +845,7 @@ static BODY *pgp_decrypt_part (BODY *a, STATE *s, FILE *fpout, BODY *p)
     unlink (pgptmpfile);
     if (s->flags & MUTT_DISPLAY)
       state_attach_puts (_("[-- Error: could not create a PGP subprocess! --]\n\n"), s);
-    return (NULL);
+    return NULL;
   }
 
   /* send the PGP passphrase to the subprocess.  Never do this if the
@@ -919,7 +919,7 @@ static BODY *pgp_decrypt_part (BODY *a, STATE *s, FILE *fpout, BODY *p)
     mutt_parse_part (fpout, tattach);
   }
 
-  return (tattach);
+  return tattach;
 }
 
 int pgp_decrypt_mime (FILE *fpin, FILE **fpout, BODY *b, BODY **cur)
@@ -957,7 +957,7 @@ int pgp_decrypt_mime (FILE *fpin, FILE **fpout, BODY *b, BODY **cur)
     if ((decoded_fp = safe_fopen (tempfile, "w+")) == NULL)
     {
       mutt_perror (tempfile);
-      return (-1);
+      return -1;
     }
     unlink (tempfile);
 
@@ -1085,7 +1085,7 @@ BODY *pgp_sign_message (BODY *a)
   mutt_mktemp (sigfile, sizeof (sigfile));
   if ((fp = safe_fopen (sigfile, "w")) == NULL)
   {
-    return (NULL);
+    return NULL;
   }
 
   mutt_mktemp (signedfile, sizeof (signedfile));
@@ -1151,7 +1151,7 @@ BODY *pgp_sign_message (BODY *a)
   {
     mutt_perror ("fclose");
     unlink (sigfile);
-    return (NULL);
+    return NULL;
   }
 
   if (err)
@@ -1161,7 +1161,7 @@ BODY *pgp_sign_message (BODY *a)
     unlink (sigfile);
     /* most likely error is a bad passphrase, so automatically forget it */
     pgp_void_passphrase ();
-    return (NULL); /* fatal error while signing */
+    return NULL; /* fatal error while signing */
   }
 
   t = mutt_new_body ();
@@ -1189,7 +1189,7 @@ BODY *pgp_sign_message (BODY *a)
   t->unlink = 1; /* ok to remove this file after sending. */
   mutt_set_parameter ("name", "signature.asc", &t->parameter);
 
-  return (a);
+  return a;
 }
 
 /* This routine attempts to find the keyids of the recipients of a message.
@@ -1310,7 +1310,7 @@ char *pgp_find_keys (ADDRESS *adrlist, int oppenc_mode)
 
     mutt_free_list (&crypt_hook_list);
   }
-  return (keylist);
+  return keylist;
 }
 
 /* Warning: "a" is no longer freed in this routine, you need
@@ -1330,7 +1330,7 @@ BODY *pgp_encrypt_message (BODY *a, char *keylist, int sign)
   if ((fpout = safe_fopen (tempfile, "w+")) == NULL)
   {
     mutt_perror (tempfile);
-    return (NULL);
+    return NULL;
   }
 
   mutt_mktemp (pgperrfile, sizeof (pgperrfile));
@@ -1368,7 +1368,7 @@ BODY *pgp_encrypt_message (BODY *a, char *keylist, int sign)
     safe_fclose (&fpout);
     safe_fclose (&pgperr);
     unlink(pgpinfile);
-    return (NULL);
+    return NULL;
   }
 
   if (sign)
@@ -1409,7 +1409,7 @@ BODY *pgp_encrypt_message (BODY *a, char *keylist, int sign)
     if (sign)
       pgp_void_passphrase (); /* just in case */
     unlink (tempfile);
-    return (NULL);
+    return NULL;
   }
 
   t = mutt_new_body ();
@@ -1437,7 +1437,7 @@ BODY *pgp_encrypt_message (BODY *a, char *keylist, int sign)
   t->parts->next->unlink = 1; /* delete after sending the message */
   t->parts->next->d_filename = safe_strdup ("msg.asc"); /* non pgp/mime can save */
 
-  return (t);
+  return t;
 }
 
 BODY *pgp_traditional_encryptsign (BODY *a, int flags, char *keylist)
@@ -1801,6 +1801,6 @@ int pgp_send_menu (HEADER *msg, int *redraw)
     }
   }
 
-  return (msg->security);
+  return msg->security;
 }
 
index d65caf329f8a7cfbd3307db3bb07a6320878a6ba..479fe20e67ac04ddd2a0e9e8261df33831c75d99 100644 (file)
@@ -140,7 +140,7 @@ static const char *_mutt_fmt_pgp_command (char *dest,
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, _mutt_fmt_pgp_command, data, 0);
 
-  return (src);
+  return src;
 }
 
 static void mutt_pgp_command (char *d, size_t dlen, struct pgp_command_context *cctx, const char *fmt)
index d05b248b91260bdcb39fce94d9a2f5a80d856271..00e4347e8613afcbfb3b0d21790e97fbbaa40b3a 100644 (file)
--- a/pgpkey.c
+++ b/pgpkey.c
@@ -274,7 +274,7 @@ static const char *pgp_entry_fmt (char *dest,
     mutt_FormatString (dest, destlen, col, cols, ifstring, mutt_attach_fmt, data, 0);
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, mutt_attach_fmt, data, 0);
-  return (src);
+  return src;
 }
 
 static void pgp_entry (char *s, size_t l, MUTTMENU * menu, int num)
@@ -644,7 +644,7 @@ static pgp_key_t pgp_select_key (pgp_key_t keys,
 
   set_option (OPTNEEDREDRAW);
 
-  return (kp);
+  return kp;
 }
 
 pgp_key_t pgp_ask_for_key (char *tag, char *whatfor,
index ecf806392d8539ac97d72bcc4fbe58c576a2180d..b6746001aa708f1679e17c254ee3698f808cfa01 100644 (file)
@@ -103,7 +103,7 @@ int mutt_num_postponed (int force)
   {
      PostCount = 0;
      LastModify = 0;
-     return (0);
+     return 0;
   }
 
   if (S_ISDIR (st.st_mode))
@@ -145,7 +145,7 @@ int mutt_num_postponed (int force)
 #endif
   }
 
-  return (PostCount);
+  return PostCount;
 }
 
 void mutt_update_num_postponed (void)
@@ -247,13 +247,13 @@ int mutt_get_postponed (CONTEXT *ctx, HEADER *hdr, HEADER **cur, char *fcc, size
   int opt_delete;
 
   if (!Postponed)
-    return (-1);
+    return -1;
 
   if ((PostContext = mx_open_mailbox (Postponed, MUTT_NOSORT, NULL)) == NULL)
   {
     PostCount = 0;
     mutt_error (_("No postponed messages."));
-    return (-1);
+    return -1;
   }
 
   if (! PostContext->msgcount)
@@ -262,7 +262,7 @@ int mutt_get_postponed (CONTEXT *ctx, HEADER *hdr, HEADER **cur, char *fcc, size
     mx_close_mailbox (PostContext, NULL);
     FREE (&PostContext);
     mutt_error (_("No postponed messages."));
-    return (-1);
+    return -1;
   }
 
   if (PostContext->msgcount == 1)
@@ -274,14 +274,14 @@ int mutt_get_postponed (CONTEXT *ctx, HEADER *hdr, HEADER **cur, char *fcc, size
   {
     mx_close_mailbox (PostContext, NULL);
     FREE (&PostContext);
-    return (-1);
+    return -1;
   }
 
   if (mutt_prepare_template (NULL, PostContext, hdr, h, 0) < 0)
   {
     mx_fastclose_mailbox (PostContext);
     FREE (&PostContext);
-    return (-1);
+    return -1;
   }
 
   /* finished with this message, so delete it. */
@@ -419,7 +419,7 @@ int mutt_get_postponed (CONTEXT *ctx, HEADER *hdr, HEADER **cur, char *fcc, size
   if (option (OPTCRYPTOPPORTUNISTICENCRYPT))
     crypt_opportunistic_encrypt (hdr);
 
-  return (code);
+  return code;
 }
 
 
@@ -567,7 +567,7 @@ int mutt_prepare_template (FILE *fp, CONTEXT *ctx, HEADER *newhdr, HEADER *hdr,
   memset (&s, 0, sizeof (s));
 
   if (!fp && (msg = mx_open_message (ctx, hdr->msgno)) == NULL)
-    return (-1);
+    return -1;
 
   if (!fp) fp = msg->fp;
 
diff --git a/query.c b/query.c
index b55ce42944924c13275f9f5b197ac6c6719a4954..5c3a43d05781db928dd0ea6931f5f5de11c56035 100644 (file)
--- a/query.c
+++ b/query.c
@@ -508,12 +508,12 @@ int mutt_query_complete (char *buf, size_t buflen)
       rfc822_free_address (&tmpa);
       free_query (&results);
       mutt_clear_error ();
-      return (0);
+      return 0;
     }
     /* multiple results, choose from query menu */
     query_menu (buf, buflen, results, 1);
   }
-  return (0);
+  return 0;
 }
 
 void mutt_query_menu (char *buf, size_t buflen)
index bd38d7cb9b7228bf7ae280e2c01000751e0cec2b..48c1a3d76bec7fb0958030fdae87e3b33f0c654e 100644 (file)
@@ -150,7 +150,7 @@ ATTACHPTR **mutt_gen_attach_list (BODY *m,
   if (level == 0)
     mutt_update_tree (idx, *idxlen);
 
-  return (idx);
+  return idx;
 }
 
 /* %c = character set: convert?
@@ -384,7 +384,7 @@ const char *mutt_attach_fmt (char *dest,
     mutt_FormatString (dest, destlen, col, cols, ifstring, mutt_attach_fmt, data, 0);
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, mutt_attach_fmt, data, 0);
-  return (src);
+  return src;
 }
 
 static void attach_entry (char *b, size_t blen, MUTTMENU *menu, int num)
@@ -726,7 +726,7 @@ static int can_print (BODY *top, int tag)
          if (!mutt_can_decode (top))
          {
            mutt_error (_("I don't know how to print %s attachments!"), type);
-           return (0);
+           return 0;
          }
        }
       }
@@ -736,7 +736,7 @@ static int can_print (BODY *top, int tag)
     if (!tag)
       break;
   }
-  return (1);
+  return 1;
 }
 
 static void print_attachment_list (FILE *fp, int tag, BODY *top, STATE *state)
index 6aaece73f7092bbb84abb934e5e745c462a670fc..b37bacbeea7ba8d52af085c0f795ab612a1b5ecc 100644 (file)
--- a/recvcmd.c
+++ b/recvcmd.c
@@ -764,7 +764,7 @@ attach_reply_envelope_defaults (ENVELOPE *env, ATTACHPTR **idx, short idxlen,
     if ((flags & SENDLISTREPLY) && !env->to)
     {
       mutt_error (_("No mailing lists found!"));
-      return (-1);
+      return -1;
     }
 
     mutt_fix_reply_recipients (env);
index e75d8418bdd4574230507ed4cdce2d29c72966f0..68483a9514a70e8c9510b78d90ecdbe6759a2b1c 100644 (file)
@@ -410,7 +410,7 @@ static const char *mix_entry_fmt (char *dest,
     mutt_FormatString (dest, destlen, col, cols, ifstring, mutt_attach_fmt, data, 0);
   else if (flags & MUTT_FORMAT_OPTIONAL)
     mutt_FormatString (dest, destlen, col, cols, elsestring, mutt_attach_fmt, data, 0);
-  return (src);
+  return src;
 }
 
 
@@ -706,7 +706,7 @@ int mix_check_message (HEADER *msg)
     if (!(fqdn = mutt_fqdn (1)))
     {
       mutt_error (_("Please set the hostname variable to a proper value when using mixmaster!"));
-      return (-1);
+      return -1;
     }
 
     /* Cc and Bcc are empty at this point. */
index 2b237c6bd72ae9d4561c355e66e1911e03a500c0..73ed7ed3a52841e1be347ae0e2d55127a656d80d 100644 (file)
--- a/rfc3676.c
+++ b/rfc3676.c
@@ -314,7 +314,7 @@ int rfc3676_handler (BODY * a, STATE * s)
   flush_par (s, &fst);
 
   FREE (&buf);
-  return (0);
+  return 0;
 }
 
 /*
index 4d60ffaca1c7b8c51baeae825f255181f45abfd4..ee09d0df5d246e4d38fa2eac1955b306680fec39 100644 (file)
--- a/rfc822.c
+++ b/rfc822.c
@@ -113,7 +113,7 @@ int rfc822_remove_from_adrlist (ADDRESS **a, const char *mailbox)
     }
   }
 
-  return (rv);
+  return rv;
 }
 
 void rfc822_free_address (ADDRESS **p)
diff --git a/score.c b/score.c
index d3c53a87997c2f30f63ddaeea013754b4d19dcbf..64d02632e6232e334e5a3ef0d87e1b8da60e8b19 100644 (file)
--- a/score.c
+++ b/score.c
@@ -70,7 +70,7 @@ int mutt_parse_score (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   if (!MoreArgs (s))
   {
     strfcpy (err->data, _("score: too few arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
   pattern = buf->data;
   mutt_buffer_init (buf);
@@ -79,7 +79,7 @@ int mutt_parse_score (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   {
     FREE (&pattern);
     strfcpy (err->data, _("score: too many arguments"), err->dsize);
-    return (-1);
+    return -1;
   }
 
   /* look for an existing entry and update the value, else add it to the end
@@ -92,7 +92,7 @@ int mutt_parse_score (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
     if ((pat = mutt_pattern_comp (pattern, 0, err)) == NULL)
     {
       FREE (&pattern);
-      return (-1);
+      return -1;
     }
     ptr = safe_calloc (1, sizeof (SCORE));
     if (last)
@@ -117,7 +117,7 @@ int mutt_parse_score (BUFFER *buf, BUFFER *s, unsigned long data, BUFFER *err)
   {
     FREE (&pattern);
     strfcpy (err->data, _("Error: score: invalid number"), err->dsize);
-    return (-1);
+    return -1;
   }
   set_option (OPTNEEDRESCORE);
   return 0;
diff --git a/send.c b/send.c
index fd81ae33dd9041f52adbd095be23ac8e2881bcea..27313aa5f568d6bf36db918d4d0e1e32cebb5cc1 100644 (file)
--- a/send.c
+++ b/send.c
@@ -88,9 +88,9 @@ static int addrsrc (ADDRESS *a, ADDRESS *lst)
   for (; lst; lst = lst->next)
   {
     if (addrcmp (a, lst))
-      return (1);
+      return 1;
   }
-  return (0);
+  return 0;
 }
 
 /* removes addresses from "b" which are contained in "a" */
@@ -205,7 +205,7 @@ static int edit_address (ADDRESS **a, /* const */ char *field)
     mutt_addrlist_to_local (*a);
     rfc822_write_address (buf, sizeof (buf), *a, 0);
     if (mutt_get_field (field, buf, sizeof (buf), MUTT_ALIAS) != 0)
-      return (-1);
+      return -1;
     rfc822_free_address (a);
     *a = mutt_expand_aliases (mutt_parse_adrlist (NULL, buf));
     if ((idna_ok = mutt_addrlist_to_intl (*a, &err)) != 0)
@@ -233,7 +233,7 @@ static int edit_envelope (ENVELOPE *en, int flags)
     else
       buf[0] = 0;
     if (mutt_get_field ("Newsgroups: ", buf, sizeof (buf), 0) != 0)
-      return (-1);
+      return -1;
     FREE (&en->newsgroups);
     en->newsgroups = safe_strdup (buf);
 
@@ -242,7 +242,7 @@ static int edit_envelope (ENVELOPE *en, int flags)
     else
       buf[0] = 0;
     if (option (OPTASKFOLLOWUP) && mutt_get_field ("Followup-To: ", buf, sizeof (buf), 0) != 0)
-      return (-1);
+      return -1;
     FREE (&en->followup_to);
     en->followup_to = safe_strdup (buf);
 
@@ -251,7 +251,7 @@ static int edit_envelope (ENVELOPE *en, int flags)
     else
       buf[0] = 0;
     if (option (OPTXCOMMENTTO) && option (OPTASKXCOMMENTTO) && mutt_get_field ("X-Comment-To: ", buf, sizeof (buf), 0) != 0)
-      return (-1);
+      return -1;
     FREE (&en->x_comment_to);
     en->x_comment_to = safe_strdup (buf);
   }
@@ -259,21 +259,21 @@ static int edit_envelope (ENVELOPE *en, int flags)
 #endif
   {
     if (edit_address (&en->to, "To: ") == -1 || en->to == NULL)
-      return (-1);
+      return -1;
     if (option (OPTASKCC) && edit_address (&en->cc, "Cc: ") == -1)
-      return (-1);
+      return -1;
     if (option (OPTASKBCC) && edit_address (&en->bcc, "Bcc: ") == -1)
-      return (-1);
+      return -1;
     if (option (OPTREPLYWITHXORIG) &&
        (flags & (SENDREPLY|SENDLISTREPLY|SENDGROUPREPLY)) &&
        (edit_address (&en->from, "From: ") == -1))
-      return (-1);
+      return -1;
   }
 
   if (en->subject)
   {
     if (option (OPTFASTREPLY))
-      return (0);
+      return 0;
     else
       strfcpy (buf, en->subject, sizeof (buf));
   }
@@ -296,7 +296,7 @@ static int edit_envelope (ENVELOPE *en, int flags)
       (!buf[0] && query_quadoption (OPT_SUBJECT, _("No subject, abort?")) != MUTT_NO))
   {
     mutt_message (_("No subject, aborting."));
-    return (-1);
+    return -1;
   }
   mutt_str_replace (&en->subject, buf);
 
@@ -564,7 +564,7 @@ static int default_to (ADDRESS **to, ENVELOPE *env, int flags, int hmfupto)
        break;
 
       default:
-       return (-1); /* abort */
+       return -1; /* abort */
       }
     }
     else
@@ -573,7 +573,7 @@ static int default_to (ADDRESS **to, ENVELOPE *env, int flags, int hmfupto)
   else
     rfc822_append (to, env->from, 0);
 
-  return (0);
+  return 0;
 }
 
 int mutt_fetch_recips (ENVELOPE *out, ENVELOPE *in, int flags)
@@ -600,12 +600,12 @@ int mutt_fetch_recips (ENVELOPE *out, ENVELOPE *in, int flags)
 
     if (in->mail_followup_to && hmfupto == MUTT_YES &&
         default_to (&out->cc, in, flags & SENDLISTREPLY, hmfupto) == MUTT_ABORT)
-      return (-1); /* abort */
+      return -1; /* abort */
   }
   else
   {
     if (default_to (&out->to, in, flags & SENDGROUPREPLY, hmfupto) == MUTT_ABORT)
-      return (-1); /* abort */
+      return -1; /* abort */
 
     if ((flags & SENDGROUPREPLY) && (!in->mail_followup_to || hmfupto != MUTT_YES))
     {
@@ -767,7 +767,7 @@ envelope_defaults (ENVELOPE *env, CONTEXT *ctx, HEADER *cur, int flags)
        * a limit such that none of the tagged message are visible.
        */
       mutt_error (_("No tagged messages are visible!"));
-      return (-1);
+      return -1;
     }
   }
   else
@@ -802,7 +802,7 @@ envelope_defaults (ENVELOPE *env, CONTEXT *ctx, HEADER *cur, int flags)
     if ((flags & SENDLISTREPLY) && !env->to)
     {
       mutt_error (_("No mailing lists found!"));
-      return (-1);
+      return -1;
     }
 
     mutt_make_misc_reply_headers (env, ctx, cur, curenv);
@@ -815,7 +815,7 @@ envelope_defaults (ENVELOPE *env, CONTEXT *ctx, HEADER *cur, int flags)
       make_reference_headers (tag ? NULL : curenv, env, ctx);
   }
 
-  return (0);
+  return 0;
 }
 
 static int
@@ -832,7 +832,7 @@ generate_body (FILE *tempfp,        /* stream for outgoing message */
   if (flags & SENDREPLY)
   {
     if ((i = query_quadoption (OPT_INCLUDE, _("Include message in reply?"))) == MUTT_ABORT)
-      return (-1);
+      return -1;
 
     if (i == MUTT_YES)
     {
@@ -847,7 +847,7 @@ generate_body (FILE *tempfp,        /* stream for outgoing message */
            if (include_reply (ctx, h, tempfp) == -1)
            {
              mutt_error (_("Could not include all requested messages!"));
-             return (-1);
+             return -1;
            }
            fputc ('\n', tempfp);
          }
@@ -922,7 +922,7 @@ generate_body (FILE *tempfp,        /* stream for outgoing message */
 
   mutt_clear_error ();
 
-  return (0);
+  return 0;
 }
 
 void mutt_set_followup_to (ENVELOPE *e)
@@ -1026,7 +1026,7 @@ static ADDRESS *set_reverse_name (ENVELOPE *env)
     if (!option (OPTREVREAL))
       FREE (&tmp->personal);
   }
-  return (tmp);
+  return tmp;
 }
 
 ADDRESS *mutt_default_from (void)
@@ -1053,7 +1053,7 @@ ADDRESS *mutt_default_from (void)
     adr->mailbox = safe_strdup (NONULL(Username));
   }
 
-  return (adr);
+  return adr;
 }
 
 static int send_message (HEADER *msg)
@@ -1068,7 +1068,7 @@ static int send_message (HEADER *msg)
   /* Write out the message in MIME form. */
   mutt_mktemp (tempfile, sizeof (tempfile));
   if ((tempfp = safe_fopen (tempfile, "w")) == NULL)
-    return (-1);
+    return -1;
 
 #ifdef USE_SMTP
   old_write_bcc = option (OPTWRITEBCC);
@@ -1092,14 +1092,14 @@ static int send_message (HEADER *msg)
   {
     safe_fclose (&tempfp);
     unlink (tempfile);
-    return (-1);
+    return -1;
   }
 
   if (fclose (tempfp) != 0)
   {
     mutt_perror (tempfile);
     unlink (tempfile);
-    return (-1);
+    return -1;
   }
 
 #ifdef MIXMASTER
@@ -1120,7 +1120,7 @@ static int send_message (HEADER *msg)
   i = mutt_invoke_sendmail (msg->env->from, msg->env->to, msg->env->cc,
                            msg->env->bcc, tempfile,
                             (msg->content->encoding == ENC8BIT));
-  return (i);
+  return i;
 }
 
 /* rfc2047 encode the content-descriptions */
index dbcaf7d49d14bc16fac9692aa4a5e5d8c64e2f45..6c315b4d0435f968e15fb0f36609a68d698749ac 100644 (file)
--- a/sendlib.c
+++ b/sendlib.c
@@ -421,7 +421,7 @@ int mutt_write_mime_body (BODY *a, FILE *f)
     {
       mutt_debug (1, "mutt_write_mime_body(): no boundary parameter found!\n");
       mutt_error (_("No boundary parameter found! [report this error]"));
-      return (-1);
+      return -1;
     }
     strfcpy (boundary, p, sizeof (boundary));
 
@@ -877,7 +877,7 @@ CONTENT *mutt_get_content_info (const char *fname, BODY *b)
   {
     mutt_debug (1, "mutt_get_content_info: %s: %s (errno %d).\n",
                 fname, strerror (errno), errno);
-    return (NULL);
+    return NULL;
   }
 
   info = safe_calloc (1, sizeof (CONTENT));
@@ -1027,7 +1027,7 @@ int mutt_lookup_mime_type (BODY *att, const char *path)
     mutt_str_replace (&att->xtype, xtype);
   }
 
-  return (type);
+  return type;
 }
 
 static void transform_to_7bit (BODY *a, FILE *fpin)
@@ -1259,7 +1259,7 @@ BODY *mutt_make_message_attach (CONTEXT *ctx, HEADER *hdr, int attach_msg)
     if ((option(OPTMIMEFORWDECODE) || option(OPTFORWDECRYPT)) &&
         (hdr->security & ENCRYPT)) {
       if (!crypt_valid_passphrase(hdr->security))
-        return (NULL);
+        return NULL;
     }
   }
 
@@ -1333,7 +1333,7 @@ BODY *mutt_make_message_attach (CONTEXT *ctx, HEADER *hdr, int attach_msg)
 
   safe_fclose (&fp);
 
-  return (body);
+  return body;
 }
 
 BODY *mutt_make_file_attach (const char *path)
@@ -1376,7 +1376,7 @@ BODY *mutt_make_file_attach (const char *path)
 
   FREE(&info);
   mutt_update_encoding (att);
-  return (att);
+  return att;
 }
 
 static int get_toplevel_encoding (BODY *a)
@@ -1386,12 +1386,12 @@ static int get_toplevel_encoding (BODY *a)
   for (; a; a = a->next)
   {
     if (a->encoding == ENCBINARY)
-      return (ENCBINARY);
+      return ENCBINARY;
     else if (a->encoding == ENC8BIT)
       e = ENC8BIT;
   }
 
-  return (e);
+  return e;
 }
 
 /* check for duplicate boundary. return 1 if duplicate */
@@ -1461,7 +1461,7 @@ char *mutt_make_date (char *s, size_t len)
            Weekdays[l->tm_wday], l->tm_mday, Months[l->tm_mon],
            l->tm_year + 1900, l->tm_hour, l->tm_min, l->tm_sec,
            (int) tz / 60, (int) abs ((int) tz) % 60);
-  return (s);
+  return s;
 }
 
 /* wrapper around mutt_write_address() so we can handle very large
@@ -2338,7 +2338,7 @@ send_msg (const char *path, char **args, const char *msg, char **tempfile)
 
   mutt_unblock_signals_system (1);
 
-  return (st);
+  return st;
 }
 
 static char **
@@ -2354,7 +2354,7 @@ add_args (char **args, size_t *argslen, size_t *argsmax, ADDRESS *addr)
       args[(*argslen)++] = addr->mailbox;
     }
   }
-  return (args);
+  return args;
 }
 
 static char **
@@ -2363,7 +2363,7 @@ add_option (char **args, size_t *argslen, size_t *argsmax, char *s)
   if (*argslen == *argsmax)
     safe_realloc (&args, (*argsmax += 5) * sizeof (char *));
   args[(*argslen)++] = s;
-  return (args);
+  return args;
 }
 
 int
@@ -2523,7 +2523,7 @@ mutt_invoke_sendmail (ADDRESS *from,      /* the sender */
     i = 1;
   else
     i = -1;
-  return (i);
+  return i;
 }
 
 /* For postponing (!final) do the necessary encodings only */
@@ -2749,7 +2749,7 @@ ADDRESS *mutt_remove_duplicates (ADDRESS *addr)
     }
   }
 
-  return (top);
+  return top;
 }
 
 static void set_noconv_flags (BODY *b, short flag)
@@ -2823,7 +2823,7 @@ int mutt_write_fcc (const char *path, HEADER *hdr, const char *msgid,
   {
     mutt_debug (1, "mutt_write_fcc(): unable to open mailbox %s in append-mode, aborting.\n",
                 path);
-    return (-1);
+    return -1;
   }
 
   /* We need to add a Content-Length field to avoid problems where a line in
@@ -2836,7 +2836,7 @@ int mutt_write_fcc (const char *path, HEADER *hdr, const char *msgid,
     {
       mutt_perror (tempfile);
       mx_close_mailbox (&f, NULL);
-      return (-1);
+      return -1;
     }
     /* remember new mail status before appending message */
     need_buffy_cleanup = 1;
@@ -2851,7 +2851,7 @@ int mutt_write_fcc (const char *path, HEADER *hdr, const char *msgid,
   {
     safe_fclose(&tempfp);
     mx_close_mailbox (&f, NULL);
-    return (-1);
+    return -1;
   }
 
   /* post == 1 => postpone message. Set mode = -1 in mutt_write_rfc822_header()
index 96cbdd7f9850c267ee2da79f4a164eb3b8878d1a..ba0c4b2836b28a9ff1ac5212fbca248049afe186 100644 (file)
--- a/signal.c
+++ b/signal.c
@@ -100,7 +100,7 @@ static void sighandler (int sig)
 #ifdef USE_SLANG_CURSES
 static int mutt_intr_hook (void)
 {
-  return (-1);
+  return -1;
 }
 #endif /* USE_SLANG_CURSES */
 
diff --git a/smime.c b/smime.c
index 939f4e9a1961521005124508a7f029a058f55ece..706b82a77387f48ea34c6021dafb967625df0fad 100644 (file)
--- a/smime.c
+++ b/smime.c
@@ -137,7 +137,7 @@ int smime_valid_passphrase (void)
   if (mutt_get_password (_("Enter S/MIME passphrase:"), SmimePass, sizeof (SmimePass)) == 0)
     {
       SmimeExptime = time (NULL) + SmimeTimeout;
-      return (1);
+      return 1;
     }
   else
     SmimeExptime = 0;
@@ -286,7 +286,7 @@ static const char *_mutt_fmt_smime_command (char *dest,
     mutt_FormatString (dest, destlen, col, cols, elsestring, _mutt_fmt_smime_command,
                       data, 0);
 
-  return (src);
+  return src;
 }
 
 
@@ -912,7 +912,7 @@ char *smime_find_keys (ADDRESS *adrlist, int oppenc_mode)
 
     smime_free_key (&key);
   }
-  return (keylist);
+  return keylist;
 }
 
 
@@ -1375,7 +1375,7 @@ BODY *smime_build_smime_entity (BODY *a, char *certlist)
   if ((fpout = safe_fopen (tempfile, "w+")) == NULL)
   {
     mutt_perror (tempfile);
-    return (NULL);
+    return NULL;
   }
 
   mutt_mktemp (smimeerrfile, sizeof (smimeerrfile));
@@ -1427,7 +1427,7 @@ BODY *smime_build_smime_entity (BODY *a, char *certlist)
     mutt_unlink (smimeinfile);
     mutt_unlink (certfile);
     safe_fclose (&fpout);
-    return (NULL);
+    return NULL;
   }
 
   safe_fclose (&smimein);
@@ -1459,7 +1459,7 @@ BODY *smime_build_smime_entity (BODY *a, char *certlist)
     /* fatal error while trying to encrypt message */
     if (!err) mutt_any_key_to_continue (_("No output from OpenSSL..."));
     mutt_unlink (tempfile);
-    return (NULL);
+    return NULL;
   }
 
   t = mutt_new_body ();
@@ -1476,7 +1476,7 @@ BODY *smime_build_smime_entity (BODY *a, char *certlist)
   t->parts=0;
   t->next=0;
 
-  return (t);
+  return t;
 }
 
 
@@ -1616,7 +1616,7 @@ BODY *smime_sign_message (BODY *a )
   {
     mutt_any_key_to_continue (_("No output from OpenSSL..."));
     mutt_unlink (signedfile);
-    return (NULL); /* fatal error while signing */
+    return NULL; /* fatal error while signing */
   }
 
   t = mutt_new_body ();
@@ -1649,7 +1649,7 @@ BODY *smime_sign_message (BODY *a )
   t->encoding = ENCBASE64;
   t->unlink = 1; /* ok to remove this file after sending. */
 
-  return (a);
+  return a;
 
 }
 
@@ -2003,7 +2003,7 @@ static BODY *smime_handle_entity (BODY *m, STATE *s, FILE *outFile)
   }
   safe_fclose (&smimeerr);
 
-  return (p);
+  return p;
 }
 
 
@@ -2036,7 +2036,7 @@ int smime_decrypt_mime (FILE *fpin, FILE **fpout, BODY *b, BODY **cur)
   if ((tmpfp = safe_fopen (tempfile, "w+")) == NULL)
   {
     mutt_perror (tempfile);
-    return (-1);
+    return -1;
   }
 
   mutt_unlink (tempfile);
@@ -2273,6 +2273,6 @@ int smime_send_menu (HEADER *msg, int *redraw)
     }
   }
 
-  return (msg->security);
+  return msg->security;
 }
 
diff --git a/sort.c b/sort.c
index 82784c1cee33a51b605e59da9a55bacfc8ba1b2e..6dad5eb42544fca7ebc3aa2bfea9316dae48daed 100644 (file)
--- a/sort.c
+++ b/sort.c
@@ -106,7 +106,7 @@ const char *mutt_get_name (ADDRESS *a)
       return (mutt_addr_for_display (a));
   }
   /* don't return NULL to avoid segfault when printing/comparing */
-  return ("");
+  return "";
 }
 
 static int compare_to (const void *a, const void *b)
@@ -260,27 +260,27 @@ sort_t *mutt_get_sort_func (int method)
   switch (method & SORT_MASK)
   {
     case SORT_RECEIVED:
-      return (compare_date_received);
+      return compare_date_received;
     case SORT_ORDER:
-      return (compare_order);
+      return compare_order;
     case SORT_DATE:
-      return (compare_date_sent);
+      return compare_date_sent;
     case SORT_SUBJECT:
-      return (compare_subject);
+      return compare_subject;
     case SORT_FROM:
-      return (compare_from);
+      return compare_from;
     case SORT_SIZE:
-      return (compare_size);
+      return compare_size;
     case SORT_TO:
-      return (compare_to);
+      return compare_to;
     case SORT_SCORE:
-      return (compare_score);
+      return compare_score;
     case SORT_SPAM:
-      return (compare_spam);
+      return compare_spam;
     case SORT_LABEL:
-      return (compare_label);
+      return compare_label;
     default:
-      return (NULL);
+      return NULL;
   }
   /* not reached */
 }
index 747a0e38c6a65eadfc22335f7640c2a55694a478..7b6b4a8af28e3777176f7b10d5ec06ad482e5df1 100644 (file)
--- a/status.c
+++ b/status.c
@@ -296,7 +296,7 @@ status_format_str (char *buf, size_t buflen, size_t col, int cols, char op, cons
 
     case 0:
       *buf = 0;
-      return (src);
+      return src;
 
     default:
       snprintf (buf, buflen, "%%%s%c", prefix, op);
@@ -308,7 +308,7 @@ status_format_str (char *buf, size_t buflen, size_t col, int cols, char op, cons
   else if (flags & MUTT_FORMAT_OPTIONAL)
     _menu_status_line (buf, buflen, col, cols, menu, elsestring);
 
-  return (src);
+  return src;
 }
 
 static void _menu_status_line (char *buf, size_t buflen, size_t col, int cols, MUTTMENU *menu, const char *p)
index d61601b4de7694522533992b55f088c4084f2a98..47a6051f0971326f1343f84647546891d6429cad 100644 (file)
--- a/system.c
+++ b/system.c
@@ -39,7 +39,7 @@ int _mutt_system (const char *cmd, int flags)
   pid_t thepid;
 
   if (!cmd || !*cmd)
-    return (0);
+    return 0;
 
   /* must ignore SIGINT and SIGQUIT */
 
@@ -137,5 +137,5 @@ int _mutt_system (const char *cmd, int flags)
 
   rc = (thepid != -1) ? (WIFEXITED (rc) ? WEXITSTATUS (rc) : -1) : -1;
 
-  return (rc);
+  return rc;
 }
index 56835d469fe9cd1e7104830b3ca8ffa208de02e9..56f9798d35a0d5b188235536ed67ee5b823ceb0e 100644 (file)
--- a/thread.c
+++ b/thread.c
@@ -31,10 +31,10 @@ static int is_descendant (THREAD *a, THREAD *b)
   while (a)
   {
     if (a == b)
-      return (1);
+      return 1;
     a = a->parent;
   }
-  return (0);
+  return 0;
 }
 
 /* Determines whether to display a message's subject. */
@@ -44,11 +44,11 @@ static int need_display_subject (CONTEXT *ctx, HEADER *hdr)
 
   /* if the user disabled subject hiding, display it */
   if (!option (OPTHIDETHREADSUBJECT))
-    return (1);
+    return 1;
 
   /* if our subject is different from our parent's, display it */
   if (hdr->subject_changed)
-    return (1);
+    return 1;
 
   /* if our subject is different from that of our closest previously displayed
    * sibling, display the subject */
@@ -58,7 +58,7 @@ static int need_display_subject (CONTEXT *ctx, HEADER *hdr)
     if (hdr && VISIBLE (hdr, ctx))
     {
       if (hdr->subject_changed)
-       return (1);
+       return 1;
       else
        break;
     }
@@ -72,14 +72,14 @@ static int need_display_subject (CONTEXT *ctx, HEADER *hdr)
     if (hdr)
     {
       if (VISIBLE (hdr, ctx))
-       return (0);
+       return 0;
       else if (hdr->subject_changed)
-       return (1);
+       return 1;
     }
   }
 
   /* if we have no visible parent or previous sibling, display the subject */
-  return (1);
+  return 1;
 }
 
 static void linearize_tree (CONTEXT *ctx)
@@ -402,7 +402,7 @@ static LIST *make_subject_list (THREAD *cur, time_t *dateptr)
     cur = cur->next;
   }
 
-  return (subjects);
+  return subjects;
 }
 
 /* find the best possible match for a parent message based upon subject.
@@ -445,7 +445,7 @@ static THREAD *find_subject (CONTEXT *ctx, THREAD *cur)
     subjects = subjects->next;
     FREE (&oldlist);
   }
-  return (last);
+  return last;
 }
 
 /* remove cur and its descendants from their current location.
@@ -612,7 +612,7 @@ THREAD *mutt_sort_subthreads (THREAD *thread, int init)
    */
   Sort ^= SORT_REVERSE;
   if (!compare_threads (NULL, NULL))
-    return (thread);
+    return thread;
 
   top = thread;
 
@@ -721,7 +721,7 @@ THREAD *mutt_sort_subthreads (THREAD *thread, int init)
       {
        Sort ^= SORT_REVERSE;
        FREE (&array);
-       return (top);
+       return top;
       }
     }
 
@@ -993,11 +993,11 @@ static HEADER *find_virtual (THREAD *cur, int reverse)
   THREAD *top = NULL;
 
   if (cur->message && cur->message->virtual >= 0)
-    return (cur->message);
+    return cur->message;
 
   top = cur;
   if ((cur = cur->child) == NULL)
-    return (NULL);
+    return NULL;
 
   while (reverse && cur->next)
     cur = cur->next;
@@ -1005,7 +1005,7 @@ static HEADER *find_virtual (THREAD *cur, int reverse)
   while (true)
   {
     if (cur->message && cur->message->virtual >= 0)
-      return (cur->message);
+      return cur->message;
 
     if (cur->child)
     {
@@ -1022,7 +1022,7 @@ static HEADER *find_virtual (THREAD *cur, int reverse)
       {
        cur = cur->parent;
        if (cur == top)
-         return (NULL);
+         return NULL;
       }
       cur = reverse ? cur->prev : cur->next;
     }
@@ -1041,7 +1041,7 @@ int _mutt_aside_thread (HEADER *hdr, short dir, short subthreads)
   if ((Sort & SORT_MASK) != SORT_THREADS)
   {
     mutt_error (_("Threading is not enabled."));
-    return (hdr->virtual);
+    return hdr->virtual;
   }
 
   cur = hdr->thread;
@@ -1071,7 +1071,7 @@ int _mutt_aside_thread (HEADER *hdr, short dir, short subthreads)
     {
       cur = cur->next;
       if (!cur)
-       return (-1);
+       return -1;
       tmp = find_virtual (cur, 0);
     } while (!tmp);
   }
@@ -1081,12 +1081,12 @@ int _mutt_aside_thread (HEADER *hdr, short dir, short subthreads)
     {
       cur = cur->prev;
       if (!cur)
-       return (-1);
+       return -1;
       tmp = find_virtual (cur, 1);
     } while (!tmp);
   }
 
-  return (tmp->virtual);
+  return tmp->virtual;
 }
 
 int mutt_parent_message (CONTEXT *ctx, HEADER *hdr, int find_root)
@@ -1097,7 +1097,7 @@ int mutt_parent_message (CONTEXT *ctx, HEADER *hdr, int find_root)
   if ((Sort & SORT_MASK) != SORT_THREADS)
   {
     mutt_error (_("Threading is not enabled."));
-    return (hdr->virtual);
+    return hdr->virtual;
   }
 
   /* Root may be the current message */
@@ -1117,7 +1117,7 @@ int mutt_parent_message (CONTEXT *ctx, HEADER *hdr, int find_root)
   if (!parent)
   {
     mutt_error (_("Parent message is not available."));
-    return (-1);
+    return -1;
   }
   if (!VISIBLE (parent, ctx))
   {
@@ -1125,9 +1125,9 @@ int mutt_parent_message (CONTEXT *ctx, HEADER *hdr, int find_root)
       mutt_error (_("Root message is not visible in this limited view."));
     else
       mutt_error (_("Parent message is not visible in this limited view."));
-    return (-1);
+    return -1;
   }
-  return (parent->virtual);
+  return parent->virtual;
 }
 
 void mutt_set_virtual (CONTEXT *ctx)
@@ -1164,7 +1164,7 @@ int _mutt_traverse_thread (CONTEXT *ctx, HEADER *cur, int flag)
   if ((Sort & SORT_MASK) != SORT_THREADS && !(flag & MUTT_THREAD_GET_HIDDEN))
   {
     mutt_error (_("Threading is not enabled."));
-    return (cur->virtual);
+    return cur->virtual;
   }
 
   final = cur->virtual;
@@ -1209,13 +1209,13 @@ int _mutt_traverse_thread (CONTEXT *ctx, HEADER *cur, int flag)
   {
     /* return value depends on action requested */
     if (flag & (MUTT_THREAD_COLLAPSE | MUTT_THREAD_UNCOLLAPSE))
-      return (final);
+      return final;
     else if (flag & MUTT_THREAD_UNREAD)
       return ((old && new) ? new : (old ? old : new));
     else if (flag & MUTT_THREAD_GET_HIDDEN)
-      return (num_hidden);
+      return num_hidden;
     else if (flag & MUTT_THREAD_NEXT_UNREAD)
-      return (min_unread);
+      return min_unread;
   }
 
   while (true)
@@ -1295,15 +1295,15 @@ int _mutt_traverse_thread (CONTEXT *ctx, HEADER *cur, int flag)
 
   /* return value depends on action requested */
   if (flag & (MUTT_THREAD_COLLAPSE | MUTT_THREAD_UNCOLLAPSE))
-    return (final);
+    return final;
   else if (flag & MUTT_THREAD_UNREAD)
     return ((old && new) ? new : (old ? old : new));
   else if (flag & MUTT_THREAD_GET_HIDDEN)
-    return (num_hidden+1);
+    return num_hidden+1;
   else if (flag & MUTT_THREAD_NEXT_UNREAD)
-    return (min_unread);
+    return min_unread;
 
-  return (0);
+  return 0;
 #undef CHECK_LIMIT
 }
 
@@ -1317,7 +1317,7 @@ int mutt_messages_in_thread (CONTEXT *ctx, HEADER *hdr, int flag)
   int i, rc;
 
   if ((Sort & SORT_MASK) != SORT_THREADS || !hdr->thread)
-    return (1);
+    return 1;
 
   threads[0] = hdr->thread;
   while (threads[0]->parent)
@@ -1339,7 +1339,7 @@ int mutt_messages_in_thread (CONTEXT *ctx, HEADER *hdr, int flag)
   if (flag)
     rc += 1;
 
-  return (rc);
+  return rc;
 }