Rocco Rutte:
[apps/madmutt.git] / browser.c
index 61bf178..1dbc644 100644 (file)
--- a/browser.c
+++ b/browser.c
@@ -1,20 +1,15 @@
 /*
+ * Copyright notice from original mutt:
  * Copyright (C) 1996-2000 Michael R. Elkins <me@mutt.org>
- * 
- *     This program is free software; you can redistribute it and/or modify
- *     it under the terms of the GNU General Public License as published by
- *     the Free Software Foundation; either version 2 of the License, or
- *     (at your option) any later version.
- * 
- *     This program is distributed in the hope that it will be useful,
- *     but WITHOUT ANY WARRANTY; without even the implied warranty of
- *     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *     GNU General Public License for more details.
- * 
- *     You should have received a copy of the GNU General Public License
- *     along with this program; if not, write to the Free Software
- *     Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
- */ 
+ *
+ * This file is part of mutt-ng, see http://www.muttng.org/.
+ * It's licensed under the GNU General Public License,
+ * please see the file GPL in the top level source directory.
+ */
+
+#if HAVE_CONFIG_H
+# include "config.h"
+#endif
 
 #include "mutt.h"
 #include "mutt_curses.h"
 #ifdef USE_NNTP
 #include "nntp.h"
 #endif
+#include "sidebar.h"
+
+#include "lib/mem.h"
+#include "lib/intl.h"
+#include "lib/str.h"
+#include "lib/list.h"
 
 #include <stdlib.h>
 #include <dirent.h>
 #include <errno.h>
 
 static struct mapping_t FolderHelp[] = {
-  { N_("Exit"),  OP_EXIT },
-  { N_("Chdir"), OP_CHANGE_DIRECTORY },
-  { N_("Mask"),  OP_ENTER_MASK },
-  { N_("Help"),  OP_HELP },
-  { NULL }
+  {N_("Exit"), OP_EXIT},
+  {N_("Chdir"), OP_CHANGE_DIRECTORY},
+  {N_("Mask"), OP_ENTER_MASK},
+  {N_("Help"), OP_HELP},
+  {NULL}
 };
 
 #ifdef USE_NNTP
 static struct mapping_t FolderNewsHelp[] = {
-  { N_("Exit"),        OP_EXIT },
-  { N_("List"),        OP_TOGGLE_MAILBOXES },
-  { N_("Subscribe"),   OP_BROWSER_SUBSCRIBE },
-  { N_("Unsubscribe"), OP_BROWSER_UNSUBSCRIBE },
-  { N_("Catchup"),     OP_CATCHUP },
-  { N_("Mask"),        OP_ENTER_MASK },
-  { N_("Help"),        OP_HELP },
-  { NULL }
+  {N_("Exit"), OP_EXIT},
+  {N_("List"), OP_TOGGLE_MAILBOXES},
+  {N_("Subscribe"), OP_BROWSER_SUBSCRIBE},
+  {N_("Unsubscribe"), OP_BROWSER_UNSUBSCRIBE},
+  {N_("Catchup"), OP_CATCHUP},
+  {N_("Mask"), OP_ENTER_MASK},
+  {N_("Help"), OP_HELP},
+  {NULL}
 };
 #endif
 
-typedef struct folder_t
-{
+typedef struct folder_t {
   struct folder_file *ff;
   int num;
 } FOLDER;
@@ -74,8 +74,7 @@ static void destroy_state (struct browser_state *state)
 {
   int c;
 
-  for (c = 0; c < state->entrylen; c++)
-  {
+  for (c = 0; c < state->entrylen; c++) {
     FREE (&((state->entry)[c].name));
     FREE (&((state->entry)[c].desc));
     FREE (&((state->entry)[c].st));
@@ -120,28 +119,27 @@ static void browser_sort (struct browser_state *state)
 {
   int (*f) (const void *, const void *);
 
-  switch (BrowserSort & SORT_MASK)
-  {
-    case SORT_ORDER:
-      return;
-    case SORT_DATE:
+  switch (BrowserSort & SORT_MASK) {
+  case SORT_ORDER:
+    return;
+  case SORT_DATE:
 #ifdef USE_NNTP
-      if (option (OPTNEWS))
-       return;
+    if (option (OPTNEWS))
+      return;
 #endif
-      f = browser_compare_date;
-      break;
-    case SORT_SIZE:
+    f = browser_compare_date;
+    break;
+  case SORT_SIZE:
 #ifdef USE_NNTP
-      if (option (OPTNEWS))
-       return;
+    if (option (OPTNEWS))
+      return;
 #endif
-      f = browser_compare_size;
-      break;
-    case SORT_SUBJECT:
-    default:
-      f = browser_compare_subject;
-      break;
+    f = browser_compare_size;
+    break;
+  case SORT_SUBJECT:
+  default:
+    f = browser_compare_subject;
+    break;
   }
   qsort (state->entry, state->entrylen, sizeof (struct folder_file), f);
 }
@@ -150,19 +148,20 @@ static int link_is_dir (const char *folder, const char *path)
 {
   struct stat st;
   char fullpath[_POSIX_PATH_MAX];
-  
+
   mutt_concat_path (fullpath, folder, path, sizeof (fullpath));
-  
+
   if (stat (fullpath, &st) == 0)
     return (S_ISDIR (st.st_mode));
   else
     return 0;
 }
 
-static const char *
-folder_format_str (char *dest, size_t destlen, char op, const char *src,
-                  const char *fmt, const char *ifstring, const char *elsestring,
-                  unsigned long data, format_flag flags)
+static const char *folder_format_str (char *dest, size_t destlen, char op,
+                                      const char *src, const char *fmt,
+                                      const char *ifstring,
+                                      const char *elsestring,
+                                      unsigned long data, format_flag flags)
 {
   char fn[SHORT_STRING], tmp[SHORT_STRING], permission[11];
   char date[16], *t_fmt;
@@ -172,149 +171,151 @@ folder_format_str (char *dest, size_t destlen, char op, const char *src,
   struct group *gr;
   int optional = (flags & M_FORMAT_OPTIONAL);
 
-  switch (op)
-  {
-    case 'C':
-      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-      snprintf (dest, destlen, tmp, folder->num + 1);
-      break;
-      
-    case 'd':
-      if (folder->ff->st != NULL)
-      {
-       tnow = time (NULL);
-       t_fmt = tnow - folder->ff->st->st_mtime < 31536000 ? "%b %d %H:%M" : "%b %d  %Y";
-       strftime (date, sizeof (date), t_fmt, localtime (&folder->ff->st->st_mtime));
-       mutt_format_s (dest, destlen, fmt, date);
-      }
-      else
-       mutt_format_s (dest, destlen, fmt, "");
-      break;
-      
-    case 'f':
+  switch (op) {
+  case 'C':
+    snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+    snprintf (dest, destlen, tmp, folder->num + 1);
+    break;
+
+  case 'd':
+    if (folder->ff->st != NULL) {
+      tnow = time (NULL);
+      t_fmt =
+        tnow - folder->ff->st->st_mtime <
+        31536000 ? "%b %d %H:%M" : "%b %d  %Y";
+      strftime (date, sizeof (date), t_fmt,
+                localtime (&folder->ff->st->st_mtime));
+      mutt_format_s (dest, destlen, fmt, date);
+    }
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  case 'f':
     {
       char *s;
+
 #ifdef USE_IMAP
       if (folder->ff->imap)
-       s = NONULL (folder->ff->desc);
+        s = NONULL (folder->ff->desc);
       else
 #endif
-       s = NONULL (folder->ff->name);
+        s = NONULL (folder->ff->name);
 
       snprintf (fn, sizeof (fn), "%s%s", s,
-               folder->ff->st ? (S_ISLNK (folder->ff->st->st_mode) ? "@" :             
-                                 (S_ISDIR (folder->ff->st->st_mode) ? "/" : 
-                                  ((folder->ff->st->st_mode & S_IXUSR) != 0 ? "*" : ""))) : "");
-      
+                folder->ff->st ? (S_ISLNK (folder->ff->st->st_mode) ? "@" :
+                                  (S_ISDIR (folder->ff->st->st_mode) ? "/" :
+                                   ((folder->ff->st->st_mode & S_IXUSR) !=
+                                    0 ? "*" : ""))) : "");
+
       mutt_format_s (dest, destlen, fmt, fn);
       break;
     }
-    case 'F':
-      if (folder->ff->st != NULL)
-      {
-       snprintf (permission, sizeof (permission), "%c%c%c%c%c%c%c%c%c%c",
-                 S_ISDIR(folder->ff->st->st_mode) ? 'd' : (S_ISLNK(folder->ff->st->st_mode) ? 'l' : '-'),
-                 (folder->ff->st->st_mode & S_IRUSR) != 0 ? 'r': '-',
-                 (folder->ff->st->st_mode & S_IWUSR) != 0 ? 'w' : '-',
-                 (folder->ff->st->st_mode & S_ISUID) != 0 ? 's' : (folder->ff->st->st_mode & S_IXUSR) != 0 ? 'x': '-',
-                 (folder->ff->st->st_mode & S_IRGRP) != 0 ? 'r' : '-',
-                 (folder->ff->st->st_mode & S_IWGRP) != 0 ? 'w' : '-',
-                 (folder->ff->st->st_mode & S_ISGID) != 0 ? 's' : (folder->ff->st->st_mode & S_IXGRP) != 0 ? 'x': '-',
-                 (folder->ff->st->st_mode & S_IROTH) != 0 ? 'r' : '-',
-                 (folder->ff->st->st_mode & S_IWOTH) != 0 ? 'w' : '-',
-                 (folder->ff->st->st_mode & S_ISVTX) != 0 ? 't' : (folder->ff->st->st_mode & S_IXOTH) != 0 ? 'x': '-');
-       mutt_format_s (dest, destlen, fmt, permission);
-      }
+  case 'F':
+    if (folder->ff->st != NULL) {
+      snprintf (permission, sizeof (permission), "%c%c%c%c%c%c%c%c%c%c",
+                S_ISDIR (folder->ff->st->
+                         st_mode) ? 'd' : (S_ISLNK (folder->ff->st->
+                                                    st_mode) ? 'l' : '-'),
+                (folder->ff->st->st_mode & S_IRUSR) != 0 ? 'r' : '-',
+                (folder->ff->st->st_mode & S_IWUSR) != 0 ? 'w' : '-',
+                (folder->ff->st->st_mode & S_ISUID) !=
+                0 ? 's' : (folder->ff->st->st_mode & S_IXUSR) !=
+                0 ? 'x' : '-',
+                (folder->ff->st->st_mode & S_IRGRP) != 0 ? 'r' : '-',
+                (folder->ff->st->st_mode & S_IWGRP) != 0 ? 'w' : '-',
+                (folder->ff->st->st_mode & S_ISGID) !=
+                0 ? 's' : (folder->ff->st->st_mode & S_IXGRP) !=
+                0 ? 'x' : '-',
+                (folder->ff->st->st_mode & S_IROTH) != 0 ? 'r' : '-',
+                (folder->ff->st->st_mode & S_IWOTH) != 0 ? 'w' : '-',
+                (folder->ff->st->st_mode & S_ISVTX) !=
+                0 ? 't' : (folder->ff->st->st_mode & S_IXOTH) !=
+                0 ? 'x' : '-');
+      mutt_format_s (dest, destlen, fmt, permission);
+    }
 #ifdef USE_IMAP
-      else if (folder->ff->imap)
-      {
-       /* mark folders with subfolders AND mail */
-       snprintf (permission, sizeof (permission), "IMAP %c",
-                 (folder->ff->inferiors && folder->ff->selectable) ? '+' : ' ');
-       mutt_format_s (dest, destlen, fmt, permission);
-      }                                        
+    else if (folder->ff->imap) {
+      /* mark folders with subfolders AND mail */
+      snprintf (permission, sizeof (permission), "IMAP %c",
+                (folder->ff->inferiors
+                 && folder->ff->selectable) ? '+' : ' ');
+      mutt_format_s (dest, destlen, fmt, permission);
+    }
 #endif
-      else
-       mutt_format_s (dest, destlen, fmt, "");
-      break;
-      
-    case 'g':
-      if (folder->ff->st != NULL)
-      {
-       if ((gr = getgrgid (folder->ff->st->st_gid)))
-         mutt_format_s (dest, destlen, fmt, gr->gr_name);
-       else
-       {
-         snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
-         snprintf (dest, destlen, tmp, folder->ff->st->st_gid);
-       }
-      }
-      else
-       mutt_format_s (dest, destlen, fmt, "");
-      break;
-      
-    case 'l':
-      if (folder->ff->st != NULL)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, folder->ff->st->st_nlink);
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  case 'g':
+    if (folder->ff->st != NULL) {
+      if ((gr = getgrgid (folder->ff->st->st_gid)))
+        mutt_format_s (dest, destlen, fmt, gr->gr_name);
+      else {
+        snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
+        snprintf (dest, destlen, tmp, folder->ff->st->st_gid);
       }
-      else
-       mutt_format_s (dest, destlen, fmt, "");
-      break;
-      
-    case 'N':
+    }
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  case 'l':
+    if (folder->ff->st != NULL) {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp, folder->ff->st->st_nlink);
+    }
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  case 'N':
 #ifdef USE_IMAP
-      if (mx_is_imap (folder->ff->desc))
-      {
-       if (!optional)
-       {
-         snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-         snprintf (dest, destlen, tmp, folder->ff->new);
-       }
-       else if (!folder->ff->new)
-         optional = 0;
-       break;
+    if (mx_is_imap (folder->ff->desc)) {
+      if (!optional) {
+        snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+        snprintf (dest, destlen, tmp, folder->ff->new);
       }
-#endif
-      snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
-      snprintf (dest, destlen, tmp, folder->ff->new ? 'N' : ' ');
-      break;
-      
-    case 's':
-      if (folder->ff->st != NULL)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
-       snprintf (dest, destlen, tmp, (long) folder->ff->st->st_size);
-      }
-      else
-       mutt_format_s (dest, destlen, fmt, "");
+      else if (!folder->ff->new)
+        optional = 0;
       break;
-      
-    case 't':
-      snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
-      snprintf (dest, destlen, tmp, folder->ff->tagged ? '*' : ' ');
-      break;
-      
-    case 'u':
-      if (folder->ff->st != NULL)
-      {
-       if ((pw = getpwuid (folder->ff->st->st_uid)))
-         mutt_format_s (dest, destlen, fmt, pw->pw_name);
-       else
-       {
-         snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
-         snprintf (dest, destlen, tmp, folder->ff->st->st_uid);
-       }
+    }
+#endif
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    snprintf (dest, destlen, tmp, folder->ff->new ? 'N' : ' ');
+    break;
+
+  case 's':
+    if (folder->ff->st != NULL) {
+      snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
+      snprintf (dest, destlen, tmp, (long) folder->ff->st->st_size);
+    }
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  case 't':
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    snprintf (dest, destlen, tmp, folder->ff->tagged ? '*' : ' ');
+    break;
+
+  case 'u':
+    if (folder->ff->st != NULL) {
+      if ((pw = getpwuid (folder->ff->st->st_uid)))
+        mutt_format_s (dest, destlen, fmt, pw->pw_name);
+      else {
+        snprintf (tmp, sizeof (tmp), "%%%sld", fmt);
+        snprintf (dest, destlen, tmp, folder->ff->st->st_uid);
       }
-      else
-       mutt_format_s (dest, destlen, fmt, "");
-      break;
-      
-    default:
-      snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
-      snprintf (dest, destlen, tmp, op);
-      break;
+    }
+    else
+      mutt_format_s (dest, destlen, fmt, "");
+    break;
+
+  default:
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    snprintf (dest, destlen, tmp, op);
+    break;
   }
 
   if (optional)
@@ -326,123 +327,115 @@ folder_format_str (char *dest, size_t destlen, char op, const char *src,
 }
 
 #ifdef USE_NNTP
-static const char *
-newsgroup_format_str (char *dest, size_t destlen, char op, const char *src,
-                      const char *fmt, const char *ifstring, const char *elsestring,
-                      unsigned long data, format_flag flags)
+static const char *newsgroup_format_str (char *dest, size_t destlen, char op,
+                                         const char *src, const char *fmt,
+                                         const char *ifstring,
+                                         const char *elsestring,
+                                         unsigned long data,
+                                         format_flag flags)
 {
   char fn[SHORT_STRING], tmp[SHORT_STRING];
   FOLDER *folder = (FOLDER *) data;
 
-  switch (op)
-  {
-    case 'C':
-      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-      snprintf (dest, destlen, tmp, folder->num + 1);
-      break;
-      
-    case 'f':
-      strncpy (fn, folder->ff->name, sizeof(fn) - 1);
-      snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
-      snprintf (dest, destlen, tmp, fn);
-      break;
-
-    case 'N':
-      snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
-      if (folder->ff->nd->subscribed)
-       snprintf (dest, destlen, tmp, ' ');
-      else
-       snprintf (dest, destlen, tmp, folder->ff->new ? 'N' : 'u');
-      break;
-
-    case 'M':
-      snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
-      if (folder->ff->nd->deleted)
-       snprintf (dest, destlen, tmp, 'D');
-      else
-       snprintf (dest, destlen, tmp, folder->ff->nd->allowed ? ' ' : '-');
-      break;
+  switch (op) {
+  case 'C':
+    snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+    snprintf (dest, destlen, tmp, folder->num + 1);
+    break;
+
+  case 'f':
+    strncpy (fn, folder->ff->name, sizeof (fn) - 1);
+    snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+    snprintf (dest, destlen, tmp, fn);
+    break;
+
+  case 'N':
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    if (folder->ff->nd->subscribed)
+      snprintf (dest, destlen, tmp, ' ');
+    else
+      snprintf (dest, destlen, tmp, folder->ff->new ? 'N' : 'u');
+    break;
 
-    case 's':
-      if (flags & M_FORMAT_OPTIONAL)
-      {
-       if (folder->ff->nd->unread != 0)
-         mutt_FormatString (dest, destlen, ifstring, newsgroup_format_str,
-               data, flags);
-       else
-         mutt_FormatString (dest, destlen, elsestring, newsgroup_format_str,
-               data, flags);
-      }
-      else if (Context && Context->data == folder->ff->nd)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, Context->unread);
-      }
+  case 'M':
+    snprintf (tmp, sizeof (tmp), "%%%sc", fmt);
+    if (folder->ff->nd->deleted)
+      snprintf (dest, destlen, tmp, 'D');
+    else
+      snprintf (dest, destlen, tmp, folder->ff->nd->allowed ? ' ' : '-');
+    break;
+
+  case 's':
+    if (flags & M_FORMAT_OPTIONAL) {
+      if (folder->ff->nd->unread != 0)
+        mutt_FormatString (dest, destlen, ifstring, newsgroup_format_str,
+                           data, flags);
       else
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, folder->ff->nd->unread);
-      }
-      break;
+        mutt_FormatString (dest, destlen, elsestring, newsgroup_format_str,
+                           data, flags);
+    }
+    else if (Context && Context->data == folder->ff->nd) {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp, Context->unread);
+    }
+    else {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp, folder->ff->nd->unread);
+    }
+    break;
 
-    case 'n':
-      if (Context && Context->data == folder->ff->nd)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, Context->new);
-      }
-      else if (option (OPTMARKOLD) &&
-               folder->ff->nd->lastCached >= folder->ff->nd->firstMessage &&
-               folder->ff->nd->lastCached <= folder->ff->nd->lastMessage)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, folder->ff->nd->lastMessage - folder->ff->nd->lastCached);
-      }
-      else
-      {
-       snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
-       snprintf (dest, destlen, tmp, folder->ff->nd->unread);
-      }
-      break;
+  case 'n':
+    if (Context && Context->data == folder->ff->nd) {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp, Context->new);
+    }
+    else if (option (OPTMARKOLD) &&
+             folder->ff->nd->lastCached >= folder->ff->nd->firstMessage &&
+             folder->ff->nd->lastCached <= folder->ff->nd->lastMessage) {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp,
+                folder->ff->nd->lastMessage - folder->ff->nd->lastCached);
+    }
+    else {
+      snprintf (tmp, sizeof (tmp), "%%%sd", fmt);
+      snprintf (dest, destlen, tmp, folder->ff->nd->unread);
+    }
+    break;
 
-    case 'd':
-      if (folder->ff->nd->desc != NULL)
-      {
-       snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
-       snprintf (dest, destlen, tmp, folder->ff->nd->desc);
-      }
-      else
-      {
-       snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
-       snprintf (dest, destlen, tmp, "");
-      }
-      break;
+  case 'd':
+    if (folder->ff->nd->desc != NULL) {
+      snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+      snprintf (dest, destlen, tmp, folder->ff->nd->desc);
+    }
+    else {
+      snprintf (tmp, sizeof (tmp), "%%%ss", fmt);
+      snprintf (dest, destlen, tmp, "");
+    }
+    break;
   }
   return (src);
 }
 #endif /* USE_NNTP */
 
-static void add_folder (MUTTMENU *m, struct browser_state *state,
-                       const char *name, const struct stat *s,
-                       void *data, int new)
+static void add_folder (MUTTMENU * m, struct browser_state *state,
+                        const char *name, const struct stat *s,
+                        void *data, int new)
 {
-  if (state->entrylen == state->entrymax)
-  {
+  if (state->entrylen == state->entrymax) {
     /* need to allocate more space */
     safe_realloc (&state->entry,
-                 sizeof (struct folder_file) * (state->entrymax += 256));
+                  sizeof (struct folder_file) * (state->entrymax += 256));
     memset (&state->entry[state->entrylen], 0,
-           sizeof (struct folder_file) * 256);
+            sizeof (struct folder_file) * 256);
     if (m)
       m->data = state->entry;
   }
 
-  if (s != NULL)
-  {
+  if (s != NULL) {
     (state->entry)[state->entrylen].mode = s->st_mode;
     (state->entry)[state->entrylen].mtime = s->st_mtime;
     (state->entry)[state->entrylen].size = s->st_size;
-    
+
     (state->entry)[state->entrylen].st = safe_malloc (sizeof (struct stat));
     memcpy ((state->entry)[state->entrylen].st, s, sizeof (struct stat));
   }
@@ -460,11 +453,13 @@ static void add_folder (MUTTMENU *m, struct browser_state *state,
   (state->entrylen)++;
 }
 
-static void init_state (struct browser_state *state, MUTTMENU *menu)
+static void init_state (struct browser_state *state, MUTTMENU * menu)
 {
   state->entrylen = 0;
   state->entrymax = 256;
-  state->entry = (struct folder_file *) safe_calloc (state->entrymax, sizeof (struct folder_file));
+  state->entry =
+    (struct folder_file *) safe_calloc (state->entrymax,
+                                        sizeof (struct folder_file));
 #ifdef USE_IMAP
   state->imap_browse = 0;
 #endif
@@ -473,12 +468,11 @@ static void init_state (struct browser_state *state, MUTTMENU *menu)
 }
 
 /* get list of all files/newsgroups with mask */
-static int examine_directory (MUTTMENU *menu, struct browser_state *state,
-                             char *d, const char *prefix)
+static int examine_directory (MUTTMENU * menu, struct browser_state *state,
+                              char *d, const char *prefix)
 {
 #ifdef USE_NNTP
-  if (option (OPTNEWS))
-  {
+  if (option (OPTNEWS)) {
     LIST *tmp;
     NNTP_DATA *data;
     NNTP_SERVER *news = CurrentNewsSrv;
@@ -486,98 +480,90 @@ static int examine_directory (MUTTMENU *menu, struct browser_state *state,
 /*  mutt_buffy_check (0); */
     init_state (state, menu);
 
-    for (tmp = news->list; tmp; tmp = tmp->next)
-    {
-      if (!(data = (NNTP_DATA *)tmp->data))
-       continue;
+    for (tmp = news->list; tmp; tmp = tmp->next) {
+      if (!(data = (NNTP_DATA *) tmp->data))
+        continue;
       if (prefix && *prefix && strncmp (prefix, data->group,
-           strlen (prefix)) != 0)
-       continue;
+                                        mutt_strlen (prefix)) != 0)
+        continue;
       if (!((regexec (Mask.rx, data->group, 0, NULL, 0) == 0) ^ Mask.not))
-       continue;
+        continue;
       add_folder (menu, state, data->group, NULL, data, data->new);
     }
   }
   else
 #endif /* USE_NNTP */
   {
-  struct stat s;
-  DIR *dp;
-  struct dirent *de;
-  char buffer[_POSIX_PATH_MAX + SHORT_STRING];
-  BUFFY *tmp;
+    struct stat s;
+    DIR *dp;
+    struct dirent *de;
+    char buffer[_POSIX_PATH_MAX + SHORT_STRING];
+    int i = -1;
+
+    while (stat (d, &s) == -1) {
+      if (errno == ENOENT) {
+        /* The last used directory is deleted, try to use the parent dir. */
+        char *c = strrchr (d, '/');
+
+        if (c && (c > d)) {
+          *c = 0;
+          continue;
+        }
+      }
+      mutt_perror (d);
+      return (-1);
+    }
 
-  while (stat (d, &s) == -1)
-  {
-    if (errno == ENOENT)
-    {
-      /* The last used directory is deleted, try to use the parent dir. */
-      char *c = strrchr (d, '/');
+    if (!S_ISDIR (s.st_mode)) {
+      mutt_error (_("%s is not a directory."), d);
+      return (-1);
+    }
 
-      if (c && (c > d))
-      {
-       *c = 0;
-       continue;
-      }
+    mutt_buffy_check (0);
+
+    if ((dp = opendir (d)) == NULL) {
+      mutt_perror (d);
+      return (-1);
     }
-    mutt_perror (d);
-    return (-1);
-  }
 
-  if (!S_ISDIR (s.st_mode))
-  {
-    mutt_error (_("%s is not a directory."), d);
-    return (-1);
-  }
+    init_state (state, menu);
 
-  mutt_buffy_check (0);
+    while ((de = readdir (dp)) != NULL) {
+      if (mutt_strcmp (de->d_name, ".") == 0)
+        continue;               /* we don't need . */
 
-  if ((dp = opendir (d)) == NULL)
-  {
-    mutt_perror (d);
-    return (-1);
-  }
+      if (prefix && *prefix
+          && mutt_strncmp (prefix, de->d_name, mutt_strlen (prefix)) != 0)
+        continue;
+      if (!((regexec (Mask.rx, de->d_name, 0, NULL, 0) == 0) ^ Mask.not))
+        continue;
 
-  init_state (state, menu);
+      mutt_concat_path (buffer, d, de->d_name, sizeof (buffer));
+      if (lstat (buffer, &s) == -1)
+        continue;
 
-  while ((de = readdir (dp)) != NULL)
-  {
-    if (mutt_strcmp (de->d_name, ".") == 0)
-      continue;    /* we don't need . */
-    
-    if (prefix && *prefix && mutt_strncmp (prefix, de->d_name, mutt_strlen (prefix)) != 0)
-      continue;
-    if (!((regexec (Mask.rx, de->d_name, 0, NULL, 0) == 0) ^ Mask.not))
-      continue;
-
-    mutt_concat_path (buffer, d, de->d_name, sizeof (buffer));
-    if (lstat (buffer, &s) == -1)
-      continue;
-    
-    if ((! S_ISREG (s.st_mode)) && (! S_ISDIR (s.st_mode)) &&
-       (! S_ISLNK (s.st_mode)))
-      continue;
-    
-    tmp = Incoming;
-    while (tmp && mutt_strcmp (buffer, tmp->path))
-      tmp = tmp->next;
-    add_folder (menu, state, de->d_name, &s, NULL, (tmp) ? tmp->new : 0);
-  }
-  closedir (dp);
+      if ((!S_ISREG (s.st_mode)) && (!S_ISDIR (s.st_mode)) &&
+          (!S_ISLNK (s.st_mode)))
+        continue;
+
+      i = buffy_lookup (buffer);
+      add_folder (menu, state, de->d_name, &s, NULL, i >= 0 ? ((BUFFY*) Incoming->data[i])->new : 0);
+    }
+    closedir (dp);
   }
+  sidebar_draw (CurrentMenu);
   browser_sort (state);
   return 0;
 }
 
 /* get list of mailboxes/subscribed newsgroups */
-static int examine_mailboxes (MUTTMENU *menu, struct browser_state *state)
+static int examine_mailboxes (MUTTMENU * menu, struct browser_state *state)
 {
   struct stat s;
   char buffer[LONG_STRING];
 
 #ifdef USE_NNTP
-  if (option (OPTNEWS))
-  {
+  if (option (OPTNEWS)) {
     LIST *tmp;
     NNTP_DATA *data;
     NNTP_SERVER *news = CurrentNewsSrv;
@@ -585,17 +571,21 @@ static int examine_mailboxes (MUTTMENU *menu, struct browser_state *state)
 /*  mutt_buffy_check (0); */
     init_state (state, menu);
 
-    for (tmp = news->list; tmp; tmp = tmp->next)
-    {
+    for (tmp = news->list; tmp; tmp = tmp->next) {
       if ((data = (NNTP_DATA *) tmp->data) != NULL && (data->new ||
-         (data->subscribed && (!option (OPTSHOWONLYUNREAD) || data->unread))))
-      add_folder (menu, state, data->group, NULL, data, data->new);
+                                                       (data->subscribed
+                                                        &&
+                                                        (!option
+                                                         (OPTSHOWONLYUNREAD)
+                                                         || data->unread))))
+        add_folder (menu, state, data->group, NULL, data, data->new);
     }
   }
   else
 #endif
   {
-    BUFFY *tmp = Incoming;
+    int i = 0;
+    BUFFY* tmp;
 
     if (!Incoming)
       return (-1);
@@ -603,81 +593,79 @@ static int examine_mailboxes (MUTTMENU *menu, struct browser_state *state)
 
     init_state (state, menu);
 
-    do
-    {
+    for (i = 0; i < Incoming->length; i++) {
+      tmp = (BUFFY*) Incoming->data[i];
 #ifdef USE_IMAP
-      if (mx_is_imap (tmp->path))
-      {
-       add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
-       continue;
+      if (mx_is_imap (tmp->path)) {
+        add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
+        continue;
       }
 #endif
 #ifdef USE_POP
-      if (mx_is_pop (tmp->path))
-      {
-       add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
-       continue;
+      if (mx_is_pop (tmp->path)) {
+        add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
+        continue;
       }
 #endif
 #ifdef USE_NNTP
-      if (mx_is_nntp (tmp->path))
-      {
-       add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
-       continue;
+      if (mx_is_nntp (tmp->path)) {
+        add_folder (menu, state, tmp->path, NULL, NULL, tmp->new);
+        continue;
       }
 #endif
       if (lstat (tmp->path, &s) == -1)
-       continue;
+        continue;
 
-      if ((! S_ISREG (s.st_mode)) && (! S_ISDIR (s.st_mode)) &&
-         (! S_ISLNK (s.st_mode)))
-       continue;
-    
-      strfcpy (buffer, NONULL(tmp->path), sizeof (buffer));
+      if ((!S_ISREG (s.st_mode)) && (!S_ISDIR (s.st_mode)) &&
+          (!S_ISLNK (s.st_mode)))
+        continue;
+
+      strfcpy (buffer, NONULL (tmp->path), sizeof (buffer));
       mutt_pretty_mailbox (buffer);
 
       add_folder (menu, state, buffer, &s, NULL, tmp->new);
     }
-    while ((tmp = tmp->next));
   }
   browser_sort (state);
   return 0;
 }
 
-int select_file_search (MUTTMENU *menu, regex_t *re, int n)
+static int select_file_search (MUTTMENU * menu, regex_t * re, int n)
 {
 #ifdef USE_NNTP
   if (option (OPTNEWS))
-    return (regexec (re, ((struct folder_file *) menu->data)[n].desc, 0, NULL, 0));
+    return (regexec
+            (re, ((struct folder_file *) menu->data)[n].desc, 0, NULL, 0));
 #endif
-  return (regexec (re, ((struct folder_file *) menu->data)[n].name, 0, NULL, 0));
+  return (regexec
+          (re, ((struct folder_file *) menu->data)[n].name, 0, NULL, 0));
 }
 
-void folder_entry (char *s, size_t slen, MUTTMENU *menu, int num)
+static void folder_entry (char *s, size_t slen, MUTTMENU * menu, int num)
 {
   FOLDER folder;
 
   folder.ff = &((struct folder_file *) menu->data)[num];
   folder.num = num;
-  
+
 #ifdef USE_NNTP
   if (option (OPTNEWS))
-    mutt_FormatString (s, slen, NONULL(GroupFormat), newsgroup_format_str, 
-      (unsigned long) &folder, M_FORMAT_ARROWCURSOR);
+    mutt_FormatString (s, slen, NONULL (GroupFormat), newsgroup_format_str,
+                       (unsigned long) &folder, M_FORMAT_ARROWCURSOR);
   else
 #endif
-    mutt_FormatString (s, slen, NONULL(FolderFormat), folder_format_str, 
-      (unsigned long) &folder, M_FORMAT_ARROWCURSOR);
+    mutt_FormatString (s, slen, NONULL (FolderFormat), folder_format_str,
+                       (unsigned long) &folder, M_FORMAT_ARROWCURSOR);
 }
 
-static void init_menu (struct browser_state *state, MUTTMENU *menu, char *title,
-                      size_t titlelen, int buffy)
+static void init_menu (struct browser_state *state, MUTTMENU * menu,
+                       char *title, size_t titlelen, int buffy)
 {
   char path[_POSIX_PATH_MAX];
 
   menu->max = state->entrylen;
 
-  if(menu->current >= menu->max)
+  if (menu->current >= menu->max)
     menu->current = menu->max - 1;
   if (menu->current < 0)
     menu->current = 0;
@@ -687,51 +675,52 @@ static void init_menu (struct browser_state *state, MUTTMENU *menu, char *title,
   menu->tagged = 0;
 
 #ifdef USE_NNTP
-  if (option (OPTNEWS))
-  {
+  if (option (OPTNEWS)) {
     if (buffy)
       snprintf (title, titlelen, "%s", _("Subscribed newsgroups"));
     else
       snprintf (title, titlelen, _("Newsgroups on server [%s]"),
-               CurrentNewsSrv->conn->account.host);
+                CurrentNewsSrv->conn->account.host);
   }
   else
 #endif
   if (buffy)
     snprintf (title, titlelen, _("Mailboxes [%d]"), mutt_buffy_check (0));
-  else
-  {
+  else {
     strfcpy (path, LastDir, sizeof (path));
     mutt_pretty_mailbox (path);
 #ifdef USE_IMAP
-  if (state->imap_browse && option (OPTIMAPLSUB))
-    snprintf (title, titlelen, _("Subscribed [%s], File mask: %s"),
-             path, NONULL (Mask.pattern));
-  else
+    if (state->imap_browse && option (OPTIMAPLSUB))
+      snprintf (title, titlelen, _("Subscribed [%s], File mask: %s"),
+                path, NONULL (Mask.pattern));
+    else
 #endif
-    snprintf (title, titlelen, _("Directory [%s], File mask: %s"),
-             path, NONULL(Mask.pattern));
+      snprintf (title, titlelen, _("Directory [%s], File mask: %s"),
+                path, NONULL (Mask.pattern));
   }
   menu->redraw = REDRAW_FULL;
 }
 
-int file_tag (MUTTMENU *menu, int n, int m)
+static int file_tag (MUTTMENU * menu, int n, int m)
 {
-  struct folder_file *ff = &(((struct folder_file *)menu->data)[n]);
+  struct folder_file *ff = &(((struct folder_file *) menu->data)[n]);
   int ot;
-  if (S_ISDIR (ff->mode) || (S_ISLNK (ff->mode) && link_is_dir (LastDir, ff->name)))
-  {
+
+  if (S_ISDIR (ff->mode)
+      || (S_ISLNK (ff->mode) && link_is_dir (LastDir, ff->name))) {
     mutt_error _("Can't attach a directory!");
+
     return 0;
   }
-  
+
   ot = ff->tagged;
   ff->tagged = (m >= 0 ? m : !ff->tagged);
-  
+
   return ff->tagged - ot;
 }
 
-void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *numfiles)
+void _mutt_select_file (char *f, size_t flen, int flags, char ***files,
+                        int *numfiles)
 {
   char buf[_POSIX_PATH_MAX];
   char prefix[_POSIX_PATH_MAX] = "";
@@ -741,99 +730,87 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
   MUTTMENU *menu;
   struct stat st;
   int i, killPrefix = 0;
-  int multiple = (flags & M_SEL_MULTI)  ? 1 : 0;
-  int folder   = (flags & M_SEL_FOLDER) ? 1 : 0;
-  int buffy    = (flags & M_SEL_BUFFY)  ? 1 : 0;
+  int multiple = (flags & M_SEL_MULTI) ? 1 : 0;
+  int folder = (flags & M_SEL_FOLDER) ? 1 : 0;
+  int buffy = (flags & M_SEL_BUFFY) ? 1 : 0;
 
   buffy = buffy && folder;
-  
+
   memset (&state, 0, sizeof (struct browser_state));
 
   if (!folder)
     strfcpy (LastDirBackup, LastDir, sizeof (LastDirBackup));
 
 #ifdef USE_NNTP
-  if (option (OPTNEWS))
-  {
+  if (option (OPTNEWS)) {
     if (*f)
       strfcpy (prefix, f, sizeof (prefix));
-    else
-    {
+    else {
       LIST *list;
 
       /* default state for news reader mode is browse subscribed newsgroups */
       buffy = 0;
-      for (list = CurrentNewsSrv->list; list; list = list->next)
-      {
-       NNTP_DATA *data = (NNTP_DATA *) list->data;
+      for (list = CurrentNewsSrv->list; list; list = list->next) {
+        NNTP_DATA *data = (NNTP_DATA *) list->data;
 
-       if (data && data->subscribed)
-       {
-         buffy = 1;
-         break;
-       }
+        if (data && data->subscribed) {
+          buffy = 1;
+          break;
+        }
       }
     }
   }
   else
 #endif
-  if (*f)
-  {
+  if (*f) {
     mutt_expand_path (f, flen);
 #ifdef USE_IMAP
-    if (mx_is_imap (f))
-    {
+    if (mx_is_imap (f)) {
       init_state (&state, NULL);
       state.imap_browse = 1;
       imap_browse (f, &state);
       strfcpy (LastDir, state.folder, sizeof (LastDir));
     }
-    else
-    {
+    else {
 #endif
-    for (i = mutt_strlen (f) - 1; i > 0 && f[i] != '/' ; i--);
-    if (i > 0)
-    {
-      if (f[0] == '/')
-      {
-       if (i > sizeof (LastDir) - 1) i = sizeof (LastDir) - 1;
-       strncpy (LastDir, f, i);
-       LastDir[i] = 0;
+      for (i = mutt_strlen (f) - 1; i > 0 && f[i] != '/'; i--);
+      if (i > 0) {
+        if (f[0] == '/') {
+          if (i > sizeof (LastDir) - 1)
+            i = sizeof (LastDir) - 1;
+          strncpy (LastDir, f, i);
+          LastDir[i] = 0;
+        }
+        else {
+          getcwd (LastDir, sizeof (LastDir));
+          safe_strcat (LastDir, sizeof (LastDir), "/");
+          safe_strncat (LastDir, sizeof (LastDir), f, i);
+        }
       }
-      else
-      {
-       getcwd (LastDir, sizeof (LastDir));
-       strcat (LastDir, "/");  /* __STRCAT_CHECKED__ */
-       strncat (LastDir, f, i);
+      else {
+        if (f[0] == '/')
+          strcpy (LastDir, "/");        /* __STRCPY_CHECKED__ */
+        else
+          getcwd (LastDir, sizeof (LastDir));
       }
-    }
-    else
-    {
-      if (f[0] == '/')
-       strcpy (LastDir, "/");          /* __STRCPY_CHECKED__ */
-      else
-       getcwd (LastDir, sizeof (LastDir));
-    }
 
-    if (i <= 0 && f[0] != '/')
-      strfcpy (prefix, f, sizeof (prefix));
-    else
-      strfcpy (prefix, f + i + 1, sizeof (prefix));
-    killPrefix = 1;
+      if (i <= 0 && f[0] != '/')
+        strfcpy (prefix, f, sizeof (prefix));
+      else
+        strfcpy (prefix, f + i + 1, sizeof (prefix));
+      killPrefix = 1;
 #ifdef USE_IMAP
     }
 #endif
   }
-  else 
-  {
+  else {
     if (!folder)
       getcwd (LastDir, sizeof (LastDir));
     else if (!LastDir[0])
-      strfcpy (LastDir, NONULL(Maildir), sizeof (LastDir));
-    
+      strfcpy (LastDir, NONULL (Maildir), sizeof (LastDir));
+
 #ifdef USE_IMAP
-    if (!buffy && mx_is_imap (LastDir))
-    {
+    if (!buffy && mx_is_imap (LastDir)) {
       init_state (&state, NULL);
       state.imap_browse = 1;
       imap_browse (LastDir, &state);
@@ -843,8 +820,7 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
 
   *f = 0;
 
-  if (buffy)
-  {
+  if (buffy) {
     if (examine_mailboxes (NULL, &state) == -1)
       goto bail;
   }
@@ -852,8 +828,8 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
 #ifdef USE_IMAP
   if (!state.imap_browse)
 #endif
-  if (examine_directory (NULL, &state, LastDir, prefix) == -1)
-    goto bail;
+    if (examine_directory (NULL, &state, LastDir, prefix) == -1)
+      goto bail;
 
   menu = mutt_new_menu ();
   menu->menu = MENU_FOLDER;
@@ -866,552 +842,537 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
 
   menu->help = mutt_compile_help (helpstr, sizeof (helpstr), MENU_FOLDER,
 #ifdef USE_NNTP
-    (option (OPTNEWS)) ? FolderNewsHelp :
+                                  (option (OPTNEWS)) ? FolderNewsHelp :
 #endif
-    FolderHelp);
+                                  FolderHelp);
 
   init_menu (&state, menu, title, sizeof (title), buffy);
 
-  FOREVER
-  {
-    switch (i = mutt_menuLoop (menu))
-    {
-      case OP_GENERIC_SELECT_ENTRY:
+  FOREVER {
+    switch (i = mutt_menuLoop (menu)) {
+    case OP_GENERIC_SELECT_ENTRY:
+
+      if (!state.entrylen) {
+        mutt_error _("No files match the file mask");
 
-       if (!state.entrylen)
-       {
-         mutt_error _("No files match the file mask");
-         break;
-       }
+        break;
+      }
 
-        if (S_ISDIR (state.entry[menu->current].mode) ||
-           (S_ISLNK (state.entry[menu->current].mode) &&
-           link_is_dir (LastDir, state.entry[menu->current].name)) 
+      if (S_ISDIR (state.entry[menu->current].mode) ||
+          (S_ISLNK (state.entry[menu->current].mode) &&
+           link_is_dir (LastDir, state.entry[menu->current].name))
 #ifdef USE_IMAP
-           || state.entry[menu->current].inferiors
+          || state.entry[menu->current].inferiors
 #endif
-           )
-       {
-         /* make sure this isn't a MH or maildir mailbox */
-         if (buffy)
-         {
-           strfcpy (buf, state.entry[menu->current].name, sizeof (buf));
-           mutt_expand_path (buf, sizeof (buf));
-         }
+        ) {
+        /* make sure this isn't a MH or maildir mailbox */
+        if (buffy) {
+          strfcpy (buf, state.entry[menu->current].name, sizeof (buf));
+          mutt_expand_path (buf, sizeof (buf));
+        }
 #ifdef USE_IMAP
-         else if (state.imap_browse)
-         {
-            strfcpy (buf, state.entry[menu->current].name, sizeof (buf));
-         }
+        else if (state.imap_browse) {
+          strfcpy (buf, state.entry[menu->current].name, sizeof (buf));
+        }
 #endif
-         else
-           mutt_concat_path (buf, LastDir, state.entry[menu->current].name, sizeof (buf));
+        else
+          mutt_concat_path (buf, LastDir, state.entry[menu->current].name,
+                            sizeof (buf));
 
-         if ((mx_get_magic (buf) <= 0)
+        if ((mx_get_magic (buf) <= 0)
 #ifdef USE_IMAP
-           || state.entry[menu->current].inferiors
+            || state.entry[menu->current].inferiors
 #endif
-           )
-         {
-           char OldLastDir[_POSIX_PATH_MAX];
-
-           /* save the old directory */
-           strfcpy (OldLastDir, LastDir, sizeof (OldLastDir));
-
-           if (mutt_strcmp (state.entry[menu->current].name, "..") == 0)
-           {
-             if (mutt_strcmp ("..", LastDir + mutt_strlen (LastDir) - 2) == 0)
-               strcat (LastDir, "/..");        /* __STRCAT_CHECKED__ */
-             else
-             {
-               char *p = strrchr (LastDir + 1, '/');
-
-               if (p)
-                 *p = 0;
-               else
-               {
-                 if (LastDir[0] == '/')
-                   LastDir[1] = 0;
-                 else
-                   strcat (LastDir, "/..");    /* __STRCAT_CHECKED__ */
-               }
-             }
-           }
-           else if (buffy)
-           {
-             strfcpy (LastDir, state.entry[menu->current].name, sizeof (LastDir));
-             mutt_expand_path (LastDir, sizeof (LastDir));
-           }
+          ) {
+          char OldLastDir[_POSIX_PATH_MAX];
+
+          /* save the old directory */
+          strfcpy (OldLastDir, LastDir, sizeof (OldLastDir));
+
+          if (mutt_strcmp (state.entry[menu->current].name, "..") == 0) {
+            if (mutt_strcmp ("..", LastDir + mutt_strlen (LastDir) - 2) == 0)
+              strcat (LastDir, "/..");  /* __STRCAT_CHECKED__ */
+            else {
+              char *p = strrchr (LastDir + 1, '/');
+
+              if (p)
+                *p = 0;
+              else {
+                if (LastDir[0] == '/')
+                  LastDir[1] = 0;
+                else
+                  strcat (LastDir, "/..");      /* __STRCAT_CHECKED__ */
+              }
+            }
+          }
+          else if (buffy) {
+            strfcpy (LastDir, state.entry[menu->current].name,
+                     sizeof (LastDir));
+            mutt_expand_path (LastDir, sizeof (LastDir));
+          }
 #ifdef USE_IMAP
-           else if (state.imap_browse)
-           {
-             int n;
-             ciss_url_t url;
-             
-              strfcpy (LastDir, state.entry[menu->current].name,
-                sizeof (LastDir));
-             /* tack on delimiter here */
-             n = strlen (LastDir)+1;
-             
-             /* special case "" needs no delimiter */
-             url_parse_ciss (&url, state.entry[menu->current].name);
-             if (url.path &&
-                 (state.entry[menu->current].delim != '\0') &&
-                 (n < sizeof (LastDir)))
-             {
-               LastDir[n] = '\0';
-               LastDir[n-1] = state.entry[menu->current].delim;
-             }
-           }
+          else if (state.imap_browse) {
+            int n;
+            ciss_url_t url;
+
+            strfcpy (LastDir, state.entry[menu->current].name,
+                     sizeof (LastDir));
+            /* tack on delimiter here */
+            n = mutt_strlen (LastDir) + 1;
+
+            /* special case "" needs no delimiter */
+            url_parse_ciss (&url, state.entry[menu->current].name);
+            if (url.path &&
+                (state.entry[menu->current].delim != '\0') &&
+                (n < sizeof (LastDir))) {
+              LastDir[n] = '\0';
+              LastDir[n - 1] = state.entry[menu->current].delim;
+            }
+          }
 #endif
-           else
-           {
-             char tmp[_POSIX_PATH_MAX];
-             mutt_concat_path (tmp, LastDir, state.entry[menu->current].name, sizeof (tmp));
-             strfcpy (LastDir, tmp, sizeof (LastDir));
-           }
-
-           destroy_state (&state);
-           if (killPrefix)
-           {
-             prefix[0] = 0;
-             killPrefix = 0;
-           }
-           buffy = 0;
+          else {
+            char tmp[_POSIX_PATH_MAX];
+
+            mutt_concat_path (tmp, LastDir, state.entry[menu->current].name,
+                              sizeof (tmp));
+            strfcpy (LastDir, tmp, sizeof (LastDir));
+          }
+
+          destroy_state (&state);
+          if (killPrefix) {
+            prefix[0] = 0;
+            killPrefix = 0;
+          }
+          buffy = 0;
 #ifdef USE_IMAP
-           if (state.imap_browse)
-           {
-             init_state (&state, NULL);
-             state.imap_browse = 1;
-             imap_browse (LastDir, &state);
-             menu->data = state.entry;
-           }
-           else
+          if (state.imap_browse) {
+            init_state (&state, NULL);
+            state.imap_browse = 1;
+            imap_browse (LastDir, &state);
+            menu->data = state.entry;
+          }
+          else
 #endif
-           if (examine_directory (menu, &state, LastDir, prefix) == -1)
-           {
-             /* try to restore the old values */
-             strfcpy (LastDir, OldLastDir, sizeof (LastDir));
-             if (examine_directory (menu, &state, LastDir, prefix) == -1)
-             {
-               strfcpy (LastDir, NONULL(Homedir), sizeof (LastDir));
-               goto bail;
-             }
-           }
-           menu->current = 0; 
-           menu->top = 0; 
-           init_menu (&state, menu, title, sizeof (title), buffy);
-           break;
-         }
-       }
+          if (examine_directory (menu, &state, LastDir, prefix) == -1) {
+            /* try to restore the old values */
+            strfcpy (LastDir, OldLastDir, sizeof (LastDir));
+            if (examine_directory (menu, &state, LastDir, prefix) == -1) {
+              strfcpy (LastDir, NONULL (Homedir), sizeof (LastDir));
+              goto bail;
+            }
+          }
+          menu->current = 0;
+          menu->top = 0;
+          init_menu (&state, menu, title, sizeof (title), buffy);
+          break;
+        }
+      }
 
 #ifdef USE_NNTP
-       if (buffy || option (OPTNEWS)) /* news have not path */
+      if (buffy || option (OPTNEWS))    /* news have not path */
 #else
-       if (buffy)
+      if (buffy)
 #endif
-       {
-         strfcpy (f, state.entry[menu->current].name, flen);
-         mutt_expand_path (f, flen);
-       }
+      {
+        strfcpy (f, state.entry[menu->current].name, flen);
+        mutt_expand_path (f, flen);
+      }
 #ifdef USE_IMAP
-       else if (state.imap_browse)
-          strfcpy (f, state.entry[menu->current].name, flen);
+      else if (state.imap_browse)
+        strfcpy (f, state.entry[menu->current].name, flen);
 #endif
-       else
-         mutt_concat_path (f, LastDir, state.entry[menu->current].name, flen);
-
-       /* Fall through to OP_EXIT */
-
-      case OP_EXIT:
-
-       if (multiple)
-       {
-         char **tfiles;
-         int i, j;
-
-         if (menu->tagged)
-         {
-           *numfiles = menu->tagged;
-           tfiles = safe_calloc (*numfiles, sizeof (char *));
-           for (i = 0, j = 0; i < state.entrylen; i++)
-           {
-             struct folder_file ff = state.entry[i];
-             char full[_POSIX_PATH_MAX];
-             if (ff.tagged)
-             {
-               mutt_concat_path (full, LastDir, ff.name, sizeof (full));
-               mutt_expand_path (full, sizeof (full));
-               tfiles[j++] = safe_strdup (full);
-             }
-           }
-           *files = tfiles;
-         }
-         else if (f[0]) /* no tagged entries. return selected entry */
-         {
-           *numfiles = 1;
-           tfiles = safe_calloc (*numfiles, sizeof (char *));
-           mutt_expand_path (f, flen);
-           tfiles[0] = safe_strdup (f);
-           *files = tfiles;
-         }
-       }
-
-       destroy_state (&state);
-       mutt_menuDestroy (&menu);
-       goto bail;
-
-      case OP_BROWSER_TELL:
-        if(state.entrylen)
-         mutt_message("%s", state.entry[menu->current].name);
-        break;
+      else
+        mutt_concat_path (f, LastDir, state.entry[menu->current].name, flen);
+
+      /* Fall through to OP_EXIT */
+
+    case OP_EXIT:
+
+      if (multiple) {
+        char **tfiles;
+        int i, j;
+
+        if (menu->tagged) {
+          *numfiles = menu->tagged;
+          tfiles = safe_calloc (*numfiles, sizeof (char *));
+          for (i = 0, j = 0; i < state.entrylen; i++) {
+            struct folder_file ff = state.entry[i];
+            char full[_POSIX_PATH_MAX];
+
+            if (ff.tagged) {
+              mutt_concat_path (full, LastDir, ff.name, sizeof (full));
+              mutt_expand_path (full, sizeof (full));
+              tfiles[j++] = safe_strdup (full);
+            }
+          }
+          *files = tfiles;
+        }
+        else if (f[0]) {        /* no tagged entries. return selected entry */
+          *numfiles = 1;
+          tfiles = safe_calloc (*numfiles, sizeof (char *));
+          mutt_expand_path (f, flen);
+          tfiles[0] = safe_strdup (f);
+          *files = tfiles;
+        }
+      }
+
+      destroy_state (&state);
+      mutt_menuDestroy (&menu);
+      goto bail;
+
+    case OP_BROWSER_TELL:
+      if (state.entrylen)
+        mutt_message ("%s", state.entry[menu->current].name);
+      break;
 
 #ifdef USE_IMAP
-      case OP_BROWSER_TOGGLE_LSUB:
-       if (option (OPTIMAPLSUB))
-       {
-         unset_option (OPTIMAPLSUB);
-       }
-       else
-       {
-         set_option (OPTIMAPLSUB);
-       }
-       mutt_ungetch (0, OP_CHECK_NEW);
-       break;
-
-      case OP_CREATE_MAILBOX:
-       if (!state.imap_browse)
-         mutt_error (_("Create is only supported for IMAP mailboxes"));
-       else
-       {
-         imap_mailbox_create (LastDir);
-         /* TODO: find a way to detect if the new folder would appear in
-          *   this window, and insert it without starting over. */
-         destroy_state (&state);
-         init_state (&state, NULL);
-         state.imap_browse = 1;
-         imap_browse (LastDir, &state);
-         menu->data = state.entry;
-         menu->current = 0; 
-         menu->top = 0; 
-         init_menu (&state, menu, title, sizeof (title), buffy);
-         MAYBE_REDRAW (menu->redraw);
-       }
-       break;
-
-      case OP_DELETE_MAILBOX:
-       if (!state.entry[menu->current].imap)
-         mutt_error (_("Delete is only supported for IMAP mailboxes"));
-       else
-        {
-         char msg[SHORT_STRING];
-         IMAP_MBOX mx;
-         int nentry = menu->current;
-         
-         imap_parse_path (state.entry[nentry].name, &mx);
-         snprintf (msg, sizeof (msg), _("Really delete mailbox \"%s\"?"),
-            mx.mbox);
-         if (mutt_yesorno (msg, M_NO) == M_YES)
-          {
-           if (!imap_delete_mailbox (Context, mx))
-            {
-             /* free the mailbox from the browser */
-             FREE (&((state.entry)[nentry].name));
-             FREE (&((state.entry)[nentry].desc));
-             /* and move all other entries up */
-             if (nentry+1 < state.entrylen)
-               memmove (state.entry + nentry, state.entry + nentry + 1,
-                  sizeof (struct folder_file) * (state.entrylen - (nentry+1)));
-             state.entrylen--;
-             mutt_message _("Mailbox deleted.");
-             init_menu (&state, menu, title, sizeof (title), buffy);
-             MAYBE_REDRAW (menu->redraw);
-           }
-         }
-         else
-           mutt_message _("Mailbox not deleted.");
-         FREE (&mx.mbox);
+    case OP_BROWSER_TOGGLE_LSUB:
+      if (option (OPTIMAPLSUB)) {
+        unset_option (OPTIMAPLSUB);
+      }
+      else {
+        set_option (OPTIMAPLSUB);
+      }
+      mutt_ungetch (0, OP_CHECK_NEW);
+      break;
+
+    case OP_CREATE_MAILBOX:
+      if (!state.imap_browse)
+        mutt_error (_("Create is only supported for IMAP mailboxes"));
+      else {
+        imap_mailbox_create (LastDir);
+        /* TODO: find a way to detect if the new folder would appear in
+         *   this window, and insert it without starting over. */
+        destroy_state (&state);
+        init_state (&state, NULL);
+        state.imap_browse = 1;
+        imap_browse (LastDir, &state);
+        menu->data = state.entry;
+        menu->current = 0;
+        menu->top = 0;
+        init_menu (&state, menu, title, sizeof (title), buffy);
+        MAYBE_REDRAW (menu->redraw);
+      }
+      break;
+
+    case OP_RENAME_MAILBOX:
+      if (!state.entry[menu->current].imap)
+        mutt_error (_("Rename is only supported for IMAP mailboxes"));
+      else {
+        int nentry = menu->current;
+
+        if (imap_mailbox_rename (state.entry[nentry].name) >= 0) {
+          destroy_state (&state);
+          init_state (&state, NULL);
+          state.imap_browse = 1;
+          imap_browse (LastDir, &state);
+          menu->data = state.entry;
+          menu->current = 0;
+          menu->top = 0;
+          init_menu (&state, menu, title, sizeof (title), buffy);
+          MAYBE_REDRAW (menu->redraw);
         }
-        break;
+      }
+      break;
+
+    case OP_DELETE_MAILBOX:
+      if (!state.entry[menu->current].imap)
+        mutt_error (_("Delete is only supported for IMAP mailboxes"));
+      else {
+        char msg[SHORT_STRING];
+        IMAP_MBOX mx;
+        int nentry = menu->current;
+
+        imap_parse_path (state.entry[nentry].name, &mx);
+        snprintf (msg, sizeof (msg), _("Really delete mailbox \"%s\"?"),
+                  mx.mbox);
+        if (mutt_yesorno (msg, M_NO) == M_YES) {
+          if (!imap_delete_mailbox (Context, mx)) {
+            /* free the mailbox from the browser */
+            FREE (&((state.entry)[nentry].name));
+            FREE (&((state.entry)[nentry].desc));
+            /* and move all other entries up */
+            if (nentry + 1 < state.entrylen)
+              memmove (state.entry + nentry, state.entry + nentry + 1,
+                       sizeof (struct folder_file) * (state.entrylen -
+                                                      (nentry + 1)));
+            state.entrylen--;
+            mutt_message _("Mailbox deleted.");
+
+            init_menu (&state, menu, title, sizeof (title), buffy);
+            MAYBE_REDRAW (menu->redraw);
+          }
+        }
+        else
+          mutt_message _("Mailbox not deleted.");
+        FREE (&mx.mbox);
+      }
+      break;
 #endif
-      
-      case OP_CHANGE_DIRECTORY:
+
+    case OP_CHANGE_DIRECTORY:
 
 #ifdef USE_NNTP
-       if (option (OPTNEWS))
-         break;
+      if (option (OPTNEWS))
+        break;
 #endif
 
-       strfcpy (buf, LastDir, sizeof (buf));
+      strfcpy (buf, LastDir, sizeof (buf));
 #ifdef USE_IMAP
-       if (!state.imap_browse)
+      if (!state.imap_browse)
 #endif
-       {
-         /* add '/' at the end of the directory name if not already there */
-         int len=mutt_strlen(LastDir);
-         if (len && LastDir[len-1] != '/' && sizeof (buf) > len)
-           buf[len]='/';
-       }
-
-       if (mutt_get_field (_("Chdir to: "), buf, sizeof (buf), M_FILE) == 0 &&
-           buf[0])
-       {
-         buffy = 0;      
-         mutt_expand_path (buf, sizeof (buf));
+      {
+        /* add '/' at the end of the directory name if not already there */
+        int len = mutt_strlen (LastDir);
+
+        if (len && LastDir[len - 1] != '/' && sizeof (buf) > len)
+          buf[len] = '/';
+      }
+
+      if (mutt_get_field (_("Chdir to: "), buf, sizeof (buf), M_FILE) == 0 &&
+          buf[0]) {
+        buffy = 0;
+        mutt_expand_path (buf, sizeof (buf));
 #ifdef USE_IMAP
-         if (mx_is_imap (buf))
-         {
-           strfcpy (LastDir, buf, sizeof (LastDir));
-           destroy_state (&state);
-           init_state (&state, NULL);
-           state.imap_browse = 1;
-           imap_browse (LastDir, &state);
-           menu->data = state.entry;
-           menu->current = 0; 
-           menu->top = 0; 
-           init_menu (&state, menu, title, sizeof (title), buffy);
-         }
-         else
+        if (mx_is_imap (buf)) {
+          strfcpy (LastDir, buf, sizeof (LastDir));
+          destroy_state (&state);
+          init_state (&state, NULL);
+          state.imap_browse = 1;
+          imap_browse (LastDir, &state);
+          menu->data = state.entry;
+          menu->current = 0;
+          menu->top = 0;
+          init_menu (&state, menu, title, sizeof (title), buffy);
+        }
+        else
 #endif
-         if (stat (buf, &st) == 0)
-         {
-           if (S_ISDIR (st.st_mode))
-           {
-             strfcpy (LastDir, buf, sizeof (LastDir));
-             destroy_state (&state);
-             if (examine_directory (menu, &state, LastDir, prefix) == 0)
-             {
-               menu->current = 0; 
-               menu->top = 0; 
-               init_menu (&state, menu, title, sizeof (title), buffy);
-             }
-             else
-             {
-               mutt_error _("Error scanning directory.");
-               destroy_state (&state);
-               mutt_menuDestroy (&menu);
-               goto bail;
-             }
-           }
-           else
-             mutt_error (_("%s is not a directory."), buf);
-         }
-         else
-           mutt_perror (buf);
-       }
-       MAYBE_REDRAW (menu->redraw);
-       break;
-       
-      case OP_ENTER_MASK:
-
-       strfcpy (buf, NONULL(Mask.pattern), sizeof (buf));
-       if (mutt_get_field (_("File Mask: "), buf, sizeof (buf), 0) == 0)
-       {
-         regex_t *rx = (regex_t *) safe_malloc (sizeof (regex_t));
-         char *s = buf;
-         int not = 0, err;
-
-         buffy = 0;
-         /* assume that the user wants to see everything */
-         if (!buf[0])
-           strfcpy (buf, ".", sizeof (buf));
-         SKIPWS (s);
-         if (*s == '!')
-         {
-           s++;
-           SKIPWS (s);
-           not = 1;
-         }
-
-         if ((err = REGCOMP (rx, s, REG_NOSUB)) != 0)
-         {
-           regerror (err, rx, buf, sizeof (buf));
-           regfree (rx);
-           FREE (&rx);
-           mutt_error ("%s", buf);
-         }
-         else
-         {
-           mutt_str_replace (&Mask.pattern, buf);
-           regfree (Mask.rx);
-           FREE (&Mask.rx);
-           Mask.rx = rx;
-           Mask.not = not;
-
-           destroy_state (&state);
+        if (stat (buf, &st) == 0) {
+          if (S_ISDIR (st.st_mode)) {
+            destroy_state (&state);
+            if (examine_directory (menu, &state, buf, prefix) == 0)
+              strfcpy (LastDir, buf, sizeof (LastDir));
+            else {
+              mutt_error _("Error scanning directory.");
+
+              if (examine_directory (menu, &state, LastDir, prefix) == -1) {
+                mutt_menuDestroy (&menu);
+                goto bail;
+              }
+            }
+            menu->current = 0;
+            menu->top = 0;
+            init_menu (&state, menu, title, sizeof (title), buffy);
+          }
+          else
+            mutt_error (_("%s is not a directory."), buf);
+        }
+        else
+          mutt_perror (buf);
+      }
+      MAYBE_REDRAW (menu->redraw);
+      break;
+
+    case OP_ENTER_MASK:
+
+      strfcpy (buf, NONULL (Mask.pattern), sizeof (buf));
+      if (mutt_get_field (_("File Mask: "), buf, sizeof (buf), 0) == 0) {
+        regex_t *rx = (regex_t *) safe_malloc (sizeof (regex_t));
+        char *s = buf;
+        int not = 0, err;
+
+        buffy = 0;
+        /* assume that the user wants to see everything */
+        if (!buf[0])
+          strfcpy (buf, ".", sizeof (buf));
+        SKIPWS (s);
+        if (*s == '!') {
+          s++;
+          SKIPWS (s);
+          not = 1;
+        }
+
+        if ((err = REGCOMP (rx, s, REG_NOSUB)) != 0) {
+          regerror (err, rx, buf, sizeof (buf));
+          regfree (rx);
+          FREE (&rx);
+          mutt_error ("%s", buf);
+        }
+        else {
+          mutt_str_replace (&Mask.pattern, buf);
+          regfree (Mask.rx);
+          FREE (&Mask.rx);
+          Mask.rx = rx;
+          Mask.not = not;
+
+          destroy_state (&state);
 #ifdef USE_IMAP
-           if (state.imap_browse)
-           {
-             init_state (&state, NULL);
-             state.imap_browse = 1;
-             imap_browse (LastDir, &state);
-             menu->data = state.entry;
-             init_menu (&state, menu, title, sizeof (title), buffy);
-           }
-           else
+          if (state.imap_browse) {
+            init_state (&state, NULL);
+            state.imap_browse = 1;
+            imap_browse (LastDir, &state);
+            menu->data = state.entry;
+            init_menu (&state, menu, title, sizeof (title), buffy);
+          }
+          else
 #endif
-           if (examine_directory (menu, &state, LastDir, NULL) == 0)
-             init_menu (&state, menu, title, sizeof (title), buffy);
-           else
-           {
-             mutt_error _("Error scanning directory.");
-             mutt_menuDestroy (&menu);
-             goto bail;
-           }
-           killPrefix = 0;
-           if (!state.entrylen)
-           {
-             mutt_error _("No files match the file mask");
-             break;
-           }
-         }
-       }
-       MAYBE_REDRAW (menu->redraw);
-       break;
-
-      case OP_SORT:
-      case OP_SORT_REVERSE:
-
-        {
-         int resort = 1;
-         int reverse = (i == OP_SORT_REVERSE);
-         
-         switch (mutt_multi_choice ((reverse) ?
-             _("Reverse sort by (d)ate, (a)lpha, si(z)e or do(n)'t sort? ") :
-             _("Sort by (d)ate, (a)lpha, si(z)e or do(n)'t sort? "),
-             _("dazn")))
-         {
-           case -1: /* abort */
-             resort = 0;
-             break;
-
-            case 1: /* (d)ate */
-             BrowserSort = SORT_DATE;
-             break;
-
-            case 2: /* (a)lpha */
-             BrowserSort = SORT_SUBJECT;
-             break;
-
-            case 3: /* si(z)e */
-             BrowserSort = SORT_SIZE;
-             break;
-
-            case 4: /* do(n)'t sort */
-             BrowserSort = SORT_ORDER;
-             resort = 0;
-             break;
-         }
-         if (resort)
-         {
-           BrowserSort |= reverse ? SORT_REVERSE : 0;
-           browser_sort (&state);
-           menu->redraw = REDRAW_FULL;
-         }
-         break;
-       }
-
-      case OP_TOGGLE_MAILBOXES:
-       buffy = 1 - buffy;
-
-      case OP_CHECK_NEW:
-       destroy_state (&state);
-       prefix[0] = 0;
-       killPrefix = 0;
-
-       if (buffy)
-       {
-         if (examine_mailboxes (menu, &state) == -1)
-           goto bail;
-       }
+          if (examine_directory (menu, &state, LastDir, NULL) == 0)
+            init_menu (&state, menu, title, sizeof (title), buffy);
+          else {
+            mutt_error _("Error scanning directory.");
+
+            mutt_menuDestroy (&menu);
+            goto bail;
+          }
+          killPrefix = 0;
+          if (!state.entrylen) {
+            mutt_error _("No files match the file mask");
+
+            break;
+          }
+        }
+      }
+      MAYBE_REDRAW (menu->redraw);
+      break;
+
+    case OP_SORT:
+    case OP_SORT_REVERSE:
+
+      {
+        int resort = 1;
+        int reverse = (i == OP_SORT_REVERSE);
+
+        switch (mutt_multi_choice ((reverse) ?
+                                   _
+                                   ("Reverse sort by (d)ate, (a)lpha, si(z)e or do(n)'t sort? ")
+                                   :
+                                   _
+                                   ("Sort by (d)ate, (a)lpha, si(z)e or do(n)'t sort? "),
+                                   _("dazn"))) {
+        case -1:               /* abort */
+          resort = 0;
+          break;
+
+        case 1:                /* (d)ate */
+          BrowserSort = SORT_DATE;
+          break;
+
+        case 2:                /* (a)lpha */
+          BrowserSort = SORT_SUBJECT;
+          break;
+
+        case 3:                /* si(z)e */
+          BrowserSort = SORT_SIZE;
+          break;
+
+        case 4:                /* do(n)'t sort */
+          BrowserSort = SORT_ORDER;
+          resort = 0;
+          break;
+        }
+        if (resort) {
+          BrowserSort |= reverse ? SORT_REVERSE : 0;
+          browser_sort (&state);
+          menu->redraw = REDRAW_FULL;
+        }
+        break;
+      }
+
+    case OP_TOGGLE_MAILBOXES:
+      buffy = 1 - buffy;
+
+    case OP_CHECK_NEW:
+      destroy_state (&state);
+      prefix[0] = 0;
+      killPrefix = 0;
+
+      if (buffy) {
+        if (examine_mailboxes (menu, &state) == -1)
+          goto bail;
+      }
 #ifdef USE_IMAP
-       else if (mx_is_imap (LastDir))
-       {
-         init_state (&state, NULL);
-         state.imap_browse = 1;
-         imap_browse (LastDir, &state);
-         menu->data = state.entry;
-       }
+      else if (mx_is_imap (LastDir)) {
+        init_state (&state, NULL);
+        state.imap_browse = 1;
+        imap_browse (LastDir, &state);
+        menu->data = state.entry;
+      }
 #endif
-       else if (examine_directory (menu, &state, LastDir, prefix) == -1)
-         goto bail;
-       init_menu (&state, menu, title, sizeof (title), buffy);
-       break;
-
-      case OP_BUFFY_LIST:
-       mutt_buffy_list ();
-       break;
-
-      case OP_BROWSER_NEW_FILE:
-
-       snprintf (buf, sizeof (buf), "%s/", LastDir);
-       if (mutt_get_field (_("New file name: "), buf, sizeof (buf), M_FILE) == 0)
-       {
-         strfcpy (f, buf, flen);
-         destroy_state (&state);
-         mutt_menuDestroy (&menu);
-         goto bail;
-       }
-       MAYBE_REDRAW (menu->redraw);
-       break;
-
-      case OP_BROWSER_VIEW_FILE:
-       if (!state.entrylen)
-       {
-         mutt_error _("No files match the file mask");
-         break;
-       }
+      else if (examine_directory (menu, &state, LastDir, prefix) == -1)
+        goto bail;
+      init_menu (&state, menu, title, sizeof (title), buffy);
+      break;
+
+    case OP_BUFFY_LIST:
+      mutt_buffy_list ();
+      break;
+
+    case OP_BROWSER_NEW_FILE:
+
+      snprintf (buf, sizeof (buf), "%s/", LastDir);
+      if (mutt_get_field (_("New file name: "), buf, sizeof (buf), M_FILE) ==
+          0) {
+        strfcpy (f, buf, flen);
+        destroy_state (&state);
+        mutt_menuDestroy (&menu);
+        goto bail;
+      }
+      MAYBE_REDRAW (menu->redraw);
+      break;
+
+    case OP_BROWSER_VIEW_FILE:
+      if (!state.entrylen) {
+        mutt_error _("No files match the file mask");
+
+        break;
+      }
 
 #ifdef USE_IMAP
-       if (state.entry[menu->current].selectable)
-       {
-         strfcpy (f, state.entry[menu->current].name, flen);
-         destroy_state (&state);
-         mutt_menuDestroy (&menu);
-         goto bail;
-       }
-       else
+      if (state.entry[menu->current].selectable) {
+        strfcpy (f, state.entry[menu->current].name, flen);
+        destroy_state (&state);
+        mutt_menuDestroy (&menu);
+        goto bail;
+      }
+      else
 #endif
-        if (S_ISDIR (state.entry[menu->current].mode) ||
-           (S_ISLNK (state.entry[menu->current].mode) &&
-           link_is_dir (LastDir, state.entry[menu->current].name)))
-       {
-         mutt_error _("Can't view a directory");
-         break;
-       } 
-       else
-       {
-         BODY *b;
-         char buf[_POSIX_PATH_MAX];
-         
-         mutt_concat_path (buf, LastDir, state.entry[menu->current].name, sizeof (buf));
-         b = mutt_make_file_attach (buf);
-         if (b != NULL)
-         {
-           mutt_view_attachment (NULL, b, M_REGULAR, NULL, NULL, 0);
-           mutt_free_body (&b);
-           menu->redraw = REDRAW_FULL;
-         }
-         else
-           mutt_error _("Error trying to view file");
-       }
-       break;
+      if (S_ISDIR (state.entry[menu->current].mode) ||
+            (S_ISLNK (state.entry[menu->current].mode) &&
+               link_is_dir (LastDir, state.entry[menu->current].name))) {
+        mutt_error _("Can't view a directory");
+
+        break;
+      }
+      else {
+        BODY *b;
+        char buf[_POSIX_PATH_MAX];
+
+        mutt_concat_path (buf, LastDir, state.entry[menu->current].name,
+                          sizeof (buf));
+        b = mutt_make_file_attach (buf);
+        if (b != NULL) {
+          mutt_view_attachment (NULL, b, M_REGULAR, NULL, NULL, 0);
+          mutt_free_body (&b);
+          menu->redraw = REDRAW_FULL;
+        }
+        else
+          mutt_error _("Error trying to view file");
+      }
+      break;
 
 #ifdef USE_NNTP
-      case OP_CATCHUP:
-      case OP_UNCATCHUP:
-       if (option (OPTNEWS))
-       {
-         struct folder_file *f = &state.entry[menu->current];
-         NNTP_DATA *nd;
-
-         if (i == OP_CATCHUP)
-           nd = mutt_newsgroup_catchup (CurrentNewsSrv, f->name);
-         else
-           nd = mutt_newsgroup_uncatchup (CurrentNewsSrv, f->name);
-
-         if (nd)
-         {
+    case OP_CATCHUP:
+    case OP_UNCATCHUP:
+      if (option (OPTNEWS)) {
+        struct folder_file *f = &state.entry[menu->current];
+        NNTP_DATA *nd;
+
+        if (i == OP_CATCHUP)
+          nd = mutt_newsgroup_catchup (CurrentNewsSrv, f->name);
+        else
+          nd = mutt_newsgroup_uncatchup (CurrentNewsSrv, f->name);
+
+        if (nd) {
 /*         FOLDER folder;
            struct folder_file ff;
            char buffer[_POSIX_PATH_MAX + SHORT_STRING];
@@ -1426,97 +1387,90 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
                  newsgroup_format_str, (unsigned long) &folder,
                  M_FORMAT_ARROWCURSOR);
            f->desc = safe_strdup (buffer); */
-           if (menu->current + 1 < menu->max)
-             menu->current++;
-           menu->redraw = REDRAW_MOTION_RESYNCH;
-         }
-       }
-       break;
-
-      case OP_LOAD_ACTIVE:
-       if (!option (OPTNEWS))
-         break;
-
-       {
-         LIST *tmp;
-         NNTP_DATA *data;
-
-         for (tmp = CurrentNewsSrv->list; tmp; tmp = tmp->next)
-         {
-           if ((data = (NNTP_DATA *)tmp->data))
-             data->deleted = 1;
-         }
-       }
-       nntp_get_active (CurrentNewsSrv);
-
-       destroy_state (&state);
-       if (buffy)
-         examine_mailboxes (menu, &state);
-       else
-         examine_directory (menu, &state, NULL, NULL);
-       init_menu (&state, menu, title, sizeof (title), buffy);
-       break;
+          if (menu->current + 1 < menu->max)
+            menu->current++;
+          menu->redraw = REDRAW_MOTION_RESYNCH;
+        }
+      }
+      break;
+
+    case OP_LOAD_ACTIVE:
+      if (!option (OPTNEWS))
+        break;
+
+      {
+        LIST *tmp;
+        NNTP_DATA *data;
+
+        for (tmp = CurrentNewsSrv->list; tmp; tmp = tmp->next) {
+          if ((data = (NNTP_DATA *) tmp->data))
+            data->deleted = 1;
+        }
+      }
+      nntp_get_active (CurrentNewsSrv);
+
+      destroy_state (&state);
+      if (buffy)
+        examine_mailboxes (menu, &state);
+      else
+        examine_directory (menu, &state, NULL, NULL);
+      init_menu (&state, menu, title, sizeof (title), buffy);
+      break;
 #endif /* USE_NNTP */
 
 #if defined USE_IMAP || defined USE_NNTP
-      case OP_BROWSER_SUBSCRIBE:
-      case OP_BROWSER_UNSUBSCRIBE:
+    case OP_BROWSER_SUBSCRIBE:
+    case OP_BROWSER_UNSUBSCRIBE:
 #endif
 #ifdef USE_NNTP
-      case OP_SUBSCRIBE_PATTERN:
-      case OP_UNSUBSCRIBE_PATTERN:
-        if (option (OPTNEWS))
-       {
-         regex_t *rx = (regex_t *) safe_malloc (sizeof (regex_t));
-         char *s = buf;
-         int j = menu->current;
-         NNTP_DATA *nd;
-         NNTP_SERVER *news = CurrentNewsSrv;
-
-         if (i == OP_SUBSCRIBE_PATTERN || i == OP_UNSUBSCRIBE_PATTERN)
-         {
-           char tmp[STRING];
-           int err;
-
-           buf[0] = 0;
-           if (i == OP_SUBSCRIBE_PATTERN)
-             snprintf (tmp, sizeof (tmp), _("Subscribe pattern: "));
-           else
-             snprintf (tmp, sizeof (tmp), _("Unsubscribe pattern: "));
-           if (mutt_get_field (tmp, buf, sizeof (buf), 0) != 0 || !buf[0])
-           {
-             FREE (&rx);
-             break;
-           }
-
-           if ((err = REGCOMP (rx, s, REG_NOSUB)) != 0)
-           {
-             regerror (err, rx, buf, sizeof (buf));
-             regfree (rx);
-             FREE (&rx);
-             mutt_error ("%s", buf);
-             break;
-           }
-           menu->redraw = REDRAW_FULL;
-           j = 0;
-         }
-         else if (!state.entrylen)
-         {
-           mutt_error _("No newsgroups match the mask");
-           break;
-         }
-
-         for ( ; j < state.entrylen; j++)
-         {
-           struct folder_file *f = &state.entry[j];
-
-           if (i == OP_BROWSER_SUBSCRIBE || i == OP_BROWSER_UNSUBSCRIBE ||
-                 regexec (rx, f->name, 0, NULL, 0) == 0)
-           {
-             if (i == OP_BROWSER_SUBSCRIBE || i == OP_SUBSCRIBE_PATTERN)
-               nd = mutt_newsgroup_subscribe (news, f->name);
-             else
-               nd = mutt_newsgroup_unsubscribe (news, f->name);
+    case OP_SUBSCRIBE_PATTERN:
+    case OP_UNSUBSCRIBE_PATTERN:
+      if (option (OPTNEWS)) {
+        regex_t *rx = (regex_t *) safe_malloc (sizeof (regex_t));
+        char *s = buf;
+        int j = menu->current;
+        NNTP_DATA *nd;
+        NNTP_SERVER *news = CurrentNewsSrv;
+
+        if (i == OP_SUBSCRIBE_PATTERN || i == OP_UNSUBSCRIBE_PATTERN) {
+          char tmp[STRING];
+          int err;
+
+          buf[0] = 0;
+          if (i == OP_SUBSCRIBE_PATTERN)
+            snprintf (tmp, sizeof (tmp), _("Subscribe pattern: "));
+          else
+            snprintf (tmp, sizeof (tmp), _("Unsubscribe pattern: "));
+          if (mutt_get_field (tmp, buf, sizeof (buf), 0) != 0 || !buf[0]) {
+            FREE (&rx);
+            break;
+          }
+
+          if ((err = REGCOMP (rx, s, REG_NOSUB)) != 0) {
+            regerror (err, rx, buf, sizeof (buf));
+            regfree (rx);
+            FREE (&rx);
+            mutt_error ("%s", buf);
+            break;
+          }
+          menu->redraw = REDRAW_FULL;
+          j = 0;
+        }
+        else if (!state.entrylen) {
+          mutt_error _("No newsgroups match the mask");
+
+          break;
+        }
+
+        for (; j < state.entrylen; j++) {
+          struct folder_file *f = &state.entry[j];
+
+          if (i == OP_BROWSER_SUBSCRIBE || i == OP_BROWSER_UNSUBSCRIBE ||
+              regexec (rx, f->name, 0, NULL, 0) == 0) {
+            if (i == OP_BROWSER_SUBSCRIBE || i == OP_SUBSCRIBE_PATTERN)
+              nd = mutt_newsgroup_subscribe (news, f->name);
+            else
+              nd = mutt_newsgroup_unsubscribe (news, f->name);
 /*           if (nd)
              {
                FOLDER folder;
@@ -1532,59 +1486,54 @@ void _mutt_select_file (char *f, size_t flen, int flags, char ***files, int *num
                        M_FORMAT_ARROWCURSOR);
                f->desc = safe_strdup (buffer);
              } */
-           }
-           if (i == OP_BROWSER_SUBSCRIBE || i == OP_BROWSER_UNSUBSCRIBE)
-           {
-             if (menu->current + 1 < menu->max)
-               menu->current++;
-             menu->redraw = REDRAW_MOTION_RESYNCH;
-             break;
-           }
-         }
-         if (i == OP_SUBSCRIBE_PATTERN)
-         {
-           LIST *grouplist = NULL;
-
-           if (news)
-             grouplist = news->list;
-           for (; grouplist; grouplist = grouplist->next)
-           {
-             nd = (NNTP_DATA *) grouplist->data;
-             if (nd && nd->group && !nd->subscribed)
-             {
-               if (regexec (rx, nd->group, 0, NULL, 0) == 0)
-               {
-                 mutt_newsgroup_subscribe (news, nd->group);
-                 add_folder (menu, &state, nd->group, NULL, nd, nd->new);
-               }
-             }
-           }
-           init_menu (&state, menu, title, sizeof (title), buffy);
-         }
-         mutt_newsrc_update (news);
-         nntp_clear_cacheindex (news);
-         if (i != OP_BROWSER_SUBSCRIBE && i != OP_BROWSER_UNSUBSCRIBE)
-           regfree (rx);
-         FREE (&rx);
-       }
+          }
+          if (i == OP_BROWSER_SUBSCRIBE || i == OP_BROWSER_UNSUBSCRIBE) {
+            if (menu->current + 1 < menu->max)
+              menu->current++;
+            menu->redraw = REDRAW_MOTION_RESYNCH;
+            break;
+          }
+        }
+        if (i == OP_SUBSCRIBE_PATTERN) {
+          LIST *grouplist = NULL;
+
+          if (news)
+            grouplist = news->list;
+          for (; grouplist; grouplist = grouplist->next) {
+            nd = (NNTP_DATA *) grouplist->data;
+            if (nd && nd->group && !nd->subscribed) {
+              if (regexec (rx, nd->group, 0, NULL, 0) == 0) {
+                mutt_newsgroup_subscribe (news, nd->group);
+                add_folder (menu, &state, nd->group, NULL, nd, nd->new);
+              }
+            }
+          }
+          init_menu (&state, menu, title, sizeof (title), buffy);
+        }
+        mutt_newsrc_update (news);
+        nntp_clear_cacheindex (news);
+        if (i != OP_BROWSER_SUBSCRIBE && i != OP_BROWSER_UNSUBSCRIBE)
+          regfree (rx);
+        FREE (&rx);
+      }
 #ifdef USE_IMAP
-       else
+      else
 #endif /* USE_IMAP && USE_NNTP */
 #endif /* USE_NNTP */
 #ifdef USE_IMAP
-       {
-         if (i == OP_BROWSER_SUBSCRIBE)
-           imap_subscribe (state.entry[menu->current].name, 1);
-         else
-           imap_subscribe (state.entry[menu->current].name, 0);
-       }
+      {
+        if (i == OP_BROWSER_SUBSCRIBE)
+          imap_subscribe (state.entry[menu->current].name, 1);
+        else
+          imap_subscribe (state.entry[menu->current].name, 0);
+      }
 #endif /* USE_IMAP */
     }
   }
-  
-  bail:
-  
+
+bail:
+
   if (!folder)
     strfcpy (LastDir, LastDirBackup, sizeof (LastDir));
-  
+
 }