<<< Date Index >>>     <<< Thread Index >>>

Re: [PATCH] runtime configurable buffy size



On Tue, Mar 20, 2007 at 10:47:55AM -0700, Brendan Cully wrote:
> > Hm, can't think of anything. The conditional is fine with me, do you want
> > me to send another patch?
> 
> That'd be great.

Ok, attached is the fixed patch.

-- 
Miroslav Lichvar
diff -r 5681216fa395 buffy.c
--- a/buffy.c   Mon Mar 19 08:12:25 2007 -0700
+++ b/buffy.c   Tue Mar 20 19:15:48 2007 +0100
@@ -45,8 +45,6 @@ static short BuffyCount = 0;  /* how many
 static short BuffyCount = 0;   /* how many boxes with new mail */
 static short BuffyNotify = 0;  /* # of unnotified new boxes */
 
-#ifdef BUFFY_SIZE
-
 /* Find the last message in the file. 
  * upon success return 0. If no message found - return -1 */
 
@@ -167,15 +165,12 @@ void mutt_update_mailbox (BUFFY * b)
     b->size = 0;
   return;
 }
-#endif
 
 int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER 
*err)
 {
   BUFFY **tmp,*tmp1;
   char buf[_POSIX_PATH_MAX];
-#ifdef BUFFY_SIZE
   struct stat sb;
-#endif /* BUFFY_SIZE */
 
   while (MoreArgs (s))
   {
@@ -232,31 +227,28 @@ int mutt_parse_mailboxes (BUFFER *path, 
     (*tmp)->notified = 1;
     (*tmp)->newly_created = 0;
 
-#ifdef BUFFY_SIZE
-    /* for buffy_size, it is important that if the folder is new (tested by
+    /* for check_mbox_size, it is important that if the folder is new (tested 
by
      * reading it), the size is set to 0 so that later when we check we see
-     * that it increased .  without buffy_size we probably don't care.
+     * that it increased .  without check_mbox_size we probably don't care.
      */
-    if (stat ((*tmp)->path, &sb) == 0 && !test_new_folder ((*tmp)->path))
+    if (option(OPTCHECKMBOXSIZE) &&
+       stat ((*tmp)->path, &sb) == 0 && !test_new_folder ((*tmp)->path))
     {
       /* some systems out there don't have an off_t type */
       (*tmp)->size = (long) sb.st_size;
     }
     else
       (*tmp)->size = 0;
-#endif /* BUFFY_SIZE */
   }
   return 0;
 }
 
-#ifdef BUFFY_SIZE
-/* people use buffy_size on systems where modified time attributes are BADLY
- * broken. Ignore them.
+/* people use check_mbox_size on systems where modified time attributes are 
+ * BADLY broken. Ignore them.
  */
-#define STAT_CHECK (sb.st_size > tmp->size)
-#else
-#define STAT_CHECK (sb.st_mtime > sb.st_atime || (tmp->newly_created && 
sb.st_ctime == sb.st_mtime && sb.st_ctime == sb.st_atime))
-#endif /* BUFFY_SIZE */
+#define STAT_CHECK_SIZE (sb.st_size > tmp->size)
+#define STAT_CHECK_TIME (sb.st_mtime > sb.st_atime || (tmp->newly_created && 
sb.st_ctime == sb.st_mtime && sb.st_ctime == sb.st_atime))
+#define STAT_CHECK (option(OPTCHECKMBOXSIZE) ? STAT_CHECK_SIZE : 
STAT_CHECK_TIME)
 
 int mutt_buffy_check (int force)
 {
@@ -323,9 +315,7 @@ int mutt_buffy_check (int force)
        * be ready for when it does. */
       tmp->newly_created = 1;
       tmp->magic = 0;
-#ifdef BUFFY_SIZE
       tmp->size = 0;
-#endif
       continue;
     }
 #ifdef USE_IMAP
@@ -365,13 +355,11 @@ int mutt_buffy_check (int force)
          BuffyCount++;
          tmp->new = 1;
        }
-#ifdef BUFFY_SIZE
-       else
+       else if (option(OPTCHECKMBOXSIZE))
        {
          /* some other program has deleted mail from the folder */
          tmp->size = (long) sb.st_size;
        }
-#endif
        if (tmp->newly_created &&
            (sb.st_ctime != sb.st_mtime || sb.st_ctime != sb.st_atime))
          tmp->newly_created = 0;
@@ -407,10 +395,8 @@ int mutt_buffy_check (int force)
        break;
       }
     }
-#ifdef BUFFY_SIZE
-    else if (Context && Context->path)
+    else if (option(OPTCHECKMBOXSIZE) && Context && Context->path)
       tmp->size = (long) sb.st_size;   /* update the size */
-#endif
 
     if (!tmp->new)
       tmp->notified = 0;
diff -r 5681216fa395 buffy.h
--- a/buffy.h   Mon Mar 19 08:12:25 2007 -0700
+++ b/buffy.h   Tue Mar 20 18:53:29 2007 +0100
@@ -23,9 +23,7 @@ typedef struct buffy_t
 typedef struct buffy_t
 {
   char *path;
-#ifdef BUFFY_SIZE
   long size;
-#endif                         /* BUFFY_SIZE */
   struct buffy_t *next;
   short new;                   /* mailbox has new mail */
   short notified;              /* user has been notified */
@@ -39,7 +37,5 @@ WHERE short BuffyTimeout INITVAL (3);
 
 extern time_t BuffyDoneTime;   /* last time we knew for sure how much mail 
there was */
 
-#ifdef BUFFY_SIZE
 BUFFY *mutt_find_mailbox (const char *path);
 void mutt_update_mailbox (BUFFY * b);
-#endif
diff -r 5681216fa395 commands.c
--- a/commands.c        Mon Mar 19 08:12:25 2007 -0700
+++ b/commands.c        Tue Mar 20 18:59:56 2007 +0100
@@ -40,9 +40,7 @@
 #include "imap.h"
 #endif
 
-#ifdef BUFFY_SIZE
 #include "buffy.h"
-#endif
 
 #include <errno.h>
 #include <unistd.h>
@@ -703,11 +701,8 @@ int mutt_save_message (HEADER *h, int de
   char prompt[SHORT_STRING], buf[_POSIX_PATH_MAX];
   CONTEXT ctx;
   struct stat st;
-#ifdef BUFFY_SIZE
   BUFFY *tmp = NULL;
-#else
   struct utimbuf ut;
-#endif
 
   *redraw = 0;
 
@@ -842,21 +837,24 @@ int mutt_save_message (HEADER *h, int de
 
     if (need_buffy_cleanup)
     {
-#ifdef BUFFY_SIZE
-      tmp = mutt_find_mailbox (buf);
-      if (tmp && !tmp->new)
-       mutt_update_mailbox (tmp);
-#else
-      /* fix up the times so buffy won't get confused */
-      if (st.st_mtime > st.st_atime)
-      {
-       ut.actime = st.st_atime;
-       ut.modtime = time (NULL);
-       utime (buf, &ut); 
+      if (option(OPTCHECKMBOXSIZE))
+      {
+       tmp = mutt_find_mailbox (buf);
+       if (tmp && !tmp->new)
+         mutt_update_mailbox (tmp);
       }
       else
-       utime (buf, NULL);
-#endif
+      {
+       /* fix up the times so buffy won't get confused */
+       if (st.st_mtime > st.st_atime)
+       {
+         ut.actime = st.st_atime;
+         ut.modtime = time (NULL);
+         utime (buf, &ut); 
+       }
+       else
+         utime (buf, NULL);
+      }
     }
 
     mutt_clear_error ();
diff -r 5681216fa395 configure.ac
--- a/configure.ac      Mon Mar 19 08:12:25 2007 -0700
+++ b/configure.ac      Tue Mar 20 18:53:29 2007 +0100
@@ -790,11 +790,6 @@ AC_ARG_ENABLE(nfs-fix, AC_HELP_STRING([-
                   [Define if you have problems with mutt not detecting
                    new/old mailboxes over NFS.  Some NFS implementations
                    incorrectly cache the attributes of small files.])
-        fi])
-
-AC_ARG_ENABLE(buffy-size, AC_HELP_STRING([--enable-buffy-size], [Use file size 
attribute instead of access time]),
-        [if test x$enableval = xyes; then
-                AC_DEFINE(BUFFY_SIZE,1,[ Define to enable the "buffy_size" 
feature. ])
         fi])
 
 AC_ARG_ENABLE(mailtool, AC_HELP_STRING([--enable-mailtool], [Enable Sun 
mailtool attachments support]),
diff -r 5681216fa395 init.h
--- a/init.h    Mon Mar 19 08:12:25 2007 -0700
+++ b/init.h    Tue Mar 20 18:53:29 2007 +0100
@@ -350,6 +350,12 @@ struct option_t MuttVars[] = {
   ** is unset, making it easier for blind persons using Braille displays to 
   ** follow these menus.  The option is disabled by default because many 
   ** visual terminals don't permit making the cursor invisible.
+  */
+  { "check_mbox_size", DT_BOOL, R_NONE, OPTCHECKMBOXSIZE, 0 },
+  /*
+  ** .pp
+  ** When this variable is set, mutt will use file size attribute instead of
+  ** access time when checking for new mail.
   */
   { "charset",         DT_STR,  R_NONE, UL &Charset, UL 0 },
   /*
diff -r 5681216fa395 main.c
--- a/main.c    Mon Mar 19 08:12:25 2007 -0700
+++ b/main.c    Tue Mar 20 18:53:29 2007 +0100
@@ -375,11 +375,6 @@ static void show_version (void)
         );
   
   puts (
-#ifdef BUFFY_SIZE
-       "+BUFFY_SIZE "
-#else
-       "-BUFFY_SIZE "
-#endif
 #ifdef EXACT_ADDRESS
        "+EXACT_ADDRESS  "
 #else
diff -r 5681216fa395 mutt.h
--- a/mutt.h    Mon Mar 19 08:12:25 2007 -0700
+++ b/mutt.h    Tue Mar 20 18:53:29 2007 +0100
@@ -339,6 +339,7 @@ enum
   OPTBEEPNEW,
   OPTBOUNCEDELIVERED,
   OPTBRAILLEFRIENDLY,
+  OPTCHECKMBOXSIZE,
   OPTCHECKNEW,
   OPTCOLLAPSEUNREAD,
   OPTCONFIRMAPPEND,
diff -r 5681216fa395 mx.c
--- a/mx.c      Mon Mar 19 08:12:25 2007 -0700
+++ b/mx.c      Tue Mar 20 19:03:58 2007 +0100
@@ -38,9 +38,7 @@
 #include "pop.h"
 #endif
 
-#ifdef BUFFY_SIZE
 #include "buffy.h"
-#endif
 
 #ifdef USE_DOTLOCK
 #include "dotlock.h"
@@ -57,9 +55,7 @@
 #include <stdlib.h>
 #include <string.h>
 #include <ctype.h>
-#ifndef BUFFY_SIZE
 #include <utime.h>
-#endif
 
 
 #define mutt_is_spool(s)  (mutt_strcmp (Spoolfile, s) == 0)
@@ -426,9 +422,7 @@ int mx_get_magic (const char *path)
   }
   else if ((f = fopen (path, "r")) != NULL)
   {
-#ifndef BUFFY_SIZE
     struct utimbuf times;
-#endif
 
     fgets (tmp, sizeof (tmp), f);
     if (mutt_strncmp ("From ", tmp, 5) == 0)
@@ -436,15 +430,17 @@ int mx_get_magic (const char *path)
     else if (mutt_strcmp (MMDF_SEP, tmp) == 0)
       magic = M_MMDF;
     safe_fclose (&f);
-#ifndef BUFFY_SIZE
-    /* need to restore the times here, the file was not really accessed,
-     * only the type was accessed.  This is important, because detection
-     * of "new mail" depends on those times set correctly.
-     */
-    times.actime = st.st_atime;
-    times.modtime = st.st_mtime;
-    utime (path, &times);
-#endif
+
+    if (!option(OPTCHECKMBOXSIZE))
+    {
+      /* need to restore the times here, the file was not really accessed,
+       * only the type was accessed.  This is important, because detection
+       * of "new mail" depends on those times set correctly.
+       */
+      times.actime = st.st_atime;
+      times.modtime = st.st_mtime;
+      utime (path, &times);
+    }
   }
   else
   {
@@ -781,9 +777,7 @@ void mx_fastclose_mailbox (CONTEXT *ctx)
 /* save changes to disk */
 static int sync_mailbox (CONTEXT *ctx, int *index_hint)
 {
-#ifdef BUFFY_SIZE
   BUFFY *tmp = NULL;
-#endif
   int rc = -1;
 
   if (!ctx->quiet)
@@ -794,9 +788,8 @@ static int sync_mailbox (CONTEXT *ctx, i
     case M_MBOX:
     case M_MMDF:
       rc = mbox_sync_mailbox (ctx, index_hint);
-#ifdef BUFFY_SIZE
-      tmp = mutt_find_mailbox (ctx->path);
-#endif
+      if (option(OPTCHECKMBOXSIZE))
+       tmp = mutt_find_mailbox (ctx->path);
       break;
       
     case M_MH:
@@ -823,10 +816,8 @@ static int sync_mailbox (CONTEXT *ctx, i
     mutt_error ( _("Could not synchronize mailbox %s!"), ctx->path);
 #endif
   
-#ifdef BUFFY_SIZE
   if (tmp && tmp->new == 0)
     mutt_update_mailbox (tmp);
-#endif
   return rc;
 }