projects
/
apps
/
madmutt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Rocco Rutte:
[apps/madmutt.git]
/
buffy.c
diff --git
a/buffy.c
b/buffy.c
index
37f479d
..
e486aba
100644
(file)
--- a/
buffy.c
+++ b/
buffy.c
@@
-174,25
+174,12
@@
void mutt_update_mailbox (BUFFY * b)
int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER *err)
{
int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER *err)
{
- BUFFY **tmp,*tmp1
,*last
;
+ BUFFY **tmp,*tmp1;
char buf[_POSIX_PATH_MAX];
char buf[_POSIX_PATH_MAX];
- int dup = 0;
#ifdef BUFFY_SIZE
struct stat sb;
#endif /* BUFFY_SIZE */
#ifdef BUFFY_SIZE
struct stat sb;
#endif /* BUFFY_SIZE */
- /*
- * FIXME
- * to get rid of correcting the ->prev pointers in sidebar.c,
- * correct them right here
- */
-
- /*
- * FIXME
- * if we really want to make the sort order of the sidebar
- * configurable, this has to go right here
- */
-
while (MoreArgs (s))
{
mutt_extract_token (path, s, 0);
while (MoreArgs (s))
{
mutt_extract_token (path, s, 0);
@@
-216,19
+203,12
@@
int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER *e
if(!*buf) continue;
/* simple check to avoid duplicates */
if(!*buf) continue;
/* simple check to avoid duplicates */
- dup = 0;
- for (tmp = &Incoming; *tmp && dup == 0; tmp = &((*tmp)->next))
+ for (tmp = &Incoming; *tmp; tmp = &((*tmp)->next))
{
{
- if (mutt_strcmp (buf, (*tmp)->path) == 0) {
- dup = 1;
- break;
- }
+ if (mutt_strcmp (buf, (*tmp)->path) == 0)
+ break;
}
}
- if (dup == 1)
- continue;
- tmp = &Incoming;
-
if(data == M_UNMAILBOXES)
{
if(*tmp)
if(data == M_UNMAILBOXES)
{
if(*tmp)
@@
-241,32
+221,16
@@
int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER *e
continue;
}
continue;
}
- /* loop over list while it's sorted */
- tmp1 = NULL;
- last = NULL;
- for (tmp = &Incoming; *tmp ; tmp = &((*tmp)->next)) {
- /*
- * FIXME
- * change this to get whatever sorting order
- */
- if (mutt_strcmp (buf, (*tmp)->path) < 0) {
- tmp1 = (*tmp);
- break;
- }
- last = (*tmp);
+ if (!*tmp)
+ {
+ *tmp = (BUFFY *) safe_calloc (1, sizeof (BUFFY));
+ (*tmp)->path = safe_strdup (buf);
+ (*tmp)->next = NULL;
+ /* it is tempting to set magic right here */
+ (*tmp)->magic = 0;
+
}
}
- /* we want: last -> tmp -> tmp1 */
- *tmp = (BUFFY *) safe_calloc (1, sizeof (BUFFY));
- (*tmp)->path = safe_strdup (buf);
- (*tmp)->magic = 0;
-
- /* correct pointers */
- (*tmp)->next = tmp1;
- if (last)
- last->next = (*tmp);
-
- /* left as-is */
(*tmp)->new = 0;
(*tmp)->notified = 1;
(*tmp)->newly_created = 0;
(*tmp)->new = 0;
(*tmp)->notified = 1;
(*tmp)->newly_created = 0;
@@
-297,6
+261,11
@@
int mutt_parse_mailboxes (BUFFER *path, BUFFER *s, unsigned long data, BUFFER *e
#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 (sb.st_mtime > sb.st_atime || (tmp->newly_created && sb.st_ctime == sb.st_mtime && sb.st_ctime == sb.st_atime))
#endif /* BUFFY_SIZE */
+/* values for force:
+ * 0 don't force any checks + update sidebar
+ * 1 force all checks + update sidebar
+ * 2 force all checks + _don't_ update sidebar
+ */
int mutt_buffy_check (int force)
{
BUFFY *tmp;
int mutt_buffy_check (int force)
{
BUFFY *tmp;
@@
-309,7
+278,7
@@
int mutt_buffy_check (int force)
CONTEXT *ctx;
#ifdef USE_IMAP
/* update postponed count as well, on force */
CONTEXT *ctx;
#ifdef USE_IMAP
/* update postponed count as well, on force */
- if (force)
+ if (force
!= 0
)
mutt_update_num_postponed ();
#endif
mutt_update_num_postponed ();
#endif
@@
-317,7
+286,7
@@
int mutt_buffy_check (int force)
if (!Incoming)
return 0;
now = time (NULL);
if (!Incoming)
return 0;
now = time (NULL);
- if (
!force
&& (now - BuffyTime < BuffyTimeout)
+ if (
force == 0
&& (now - BuffyTime < BuffyTimeout)
#ifdef USE_IMAP
&& (now - ImapBuffyTime < ImapBuffyTimeout))
#else
#ifdef USE_IMAP
&& (now - ImapBuffyTime < ImapBuffyTimeout))
#else
@@
-326,11
+295,11
@@
int mutt_buffy_check (int force)
return BuffyCount;
last1 = BuffyTime;
return BuffyCount;
last1 = BuffyTime;
- if (force || now - BuffyTime >= BuffyTimeout)
+ if (force
!= 0
|| now - BuffyTime >= BuffyTimeout)
BuffyTime = now;
#ifdef USE_IMAP
last2 = ImapBuffyTime;
BuffyTime = now;
#ifdef USE_IMAP
last2 = ImapBuffyTime;
- if (force || now - ImapBuffyTime >= ImapBuffyTimeout)
+ if (force
!= 0
|| now - ImapBuffyTime >= ImapBuffyTimeout)
ImapBuffyTime = now;
#endif
BuffyCount = 0;
ImapBuffyTime = now;
#endif
BuffyCount = 0;
@@
-406,13
+375,11
@@
int mutt_buffy_check (int force)
case M_MBOX:
case M_MMDF:
/* only check on force or $mail_check reached */
case M_MBOX:
case M_MMDF:
/* only check on force or $mail_check reached */
- if (force || (now - last1 >= BuffyTimeout)) {
- tmp->new = 0;
- tmp->msg_unread = 0;
+ if (force != 0 || (now - last1 >= BuffyTimeout)) {
if (SidebarWidth == 0 || !option (OPTMBOXPANE)) {
if (STAT_CHECK) {
BuffyCount++;
if (SidebarWidth == 0 || !option (OPTMBOXPANE)) {
if (STAT_CHECK) {
BuffyCount++;
- tmp->new =
tmp->has_new =
1;
+ tmp->new = 1;
}
#ifdef BUFFY_SIZE
else
}
#ifdef BUFFY_SIZE
else
@@
-424,33
+391,25
@@
int mutt_buffy_check (int force)
} else if (SidebarWidth > 0 && option (OPTMBOXPANE) &&
(STAT_CHECK || tmp->msgcount == 0)) {
/* sidebar visible */
} else if (SidebarWidth > 0 && option (OPTMBOXPANE) &&
(STAT_CHECK || tmp->msgcount == 0)) {
/* sidebar visible */
- int msg_count = 0, msg_new = 0, msg_unread = 0;
BuffyCount++;
if ((ctx = mx_open_mailbox (tmp->path, M_READONLY | M_QUIET | M_NOSORT, NULL)) != NULL) {
BuffyCount++;
if ((ctx = mx_open_mailbox (tmp->path, M_READONLY | M_QUIET | M_NOSORT, NULL)) != NULL) {
- msg_count = ctx->msgcount;
- msg_new = ctx->new;
- msg_unread = ctx->unread;
+ tmp->msgcount = ctx->msgcount;
+ tmp->new = ctx->new;
+ tmp->msg_unread = ctx->new; /* for sidebar, wtf? */
+ tmp->msg_flagged = ctx->flagged;
mx_close_mailbox (ctx, 0);
}
mx_close_mailbox (ctx, 0);
}
- tmp->msgcount = msg_count;
- tmp->new = msg_new;
- tmp->msg_unread = msg_unread;
- tmp->has_new = msg_new > 0;
}
if (tmp->newly_created &&
(sb.st_ctime != sb.st_mtime || sb.st_ctime != sb.st_atime))
tmp->newly_created = 0;
}
if (tmp->newly_created &&
(sb.st_ctime != sb.st_mtime || sb.st_ctime != sb.st_atime))
tmp->newly_created = 0;
- tmp->has_new = tmp->new > 0;
- } else if (tmp->new > 0) {
- /* keep current stats if !force and !$mail_check reached */
+ } else if (tmp->new > 0)
BuffyCount++;
BuffyCount++;
- tmp->has_new = 1;
- }
break;
case M_MAILDIR:
/* only check on force or $mail_check reached */
break;
case M_MAILDIR:
/* only check on force or $mail_check reached */
- if (force || (now - last1 >= BuffyTimeout)) {
+ if (force
!= 0
|| (now - last1 >= BuffyTimeout)) {
snprintf (path, sizeof (path), "%s/new", tmp->path);
if ((dirp = opendir (path)) == NULL)
{
snprintf (path, sizeof (path), "%s/new", tmp->path);
if ((dirp = opendir (path)) == NULL)
{
@@
-470,7
+429,7
@@
int mutt_buffy_check (int force)
if (tmp->new == 0)
{
BuffyCount++;
if (tmp->new == 0)
{
BuffyCount++;
- tmp->
has_new = tmp->
new = 1;
+ tmp->new = 1;
if (SidebarWidth == 0 || !option (OPTMBOXPANE))
/* if sidebar invisible -> done */
break;
if (SidebarWidth == 0 || !option (OPTMBOXPANE))
/* if sidebar invisible -> done */
break;
@@
-481,7
+440,6
@@
int mutt_buffy_check (int force)
}
}
closedir (dirp);
}
}
closedir (dirp);
- tmp->has_new = tmp->new > 0;
if (SidebarWidth > 0 && option (OPTMBOXPANE))
{
if (SidebarWidth > 0 && option (OPTMBOXPANE))
{
@@
-492,29
+450,27
@@
int mutt_buffy_check (int force)
tmp->magic = 0;
break;
}
tmp->magic = 0;
break;
}
+ tmp->msg_flagged = 0;
while ((de = readdir (dirp)) != NULL)
{
char *p;
while ((de = readdir (dirp)) != NULL)
{
char *p;
- if (*de->d_name != '.' &&
- (!(p = strstr (de->d_name, ":2,")) || !strchr (p + 3, 'T')))
- {
- tmp->msgcount++;
+ if (*de->d_name != '.' && (p = strstr (de->d_name, ":2,")) != NULL) {
+ if (!strchr (p + 3, 'T'))
+ tmp->msgcount++;
+ if (strchr (p + 3, 'F'))
+ tmp->msg_flagged++;
}
}
closedir (dirp);
}
}
}
closedir (dirp);
}
- } else if (tmp->new > 0)
{
+ } else if (tmp->new > 0)
/* keep current stats if !force and !$mail_check reached */
BuffyCount++;
/* keep current stats if !force and !$mail_check reached */
BuffyCount++;
- tmp->has_new = 1;
- }
break;
case M_MH:
/* only check on force or $mail_check reached */
break;
case M_MH:
/* only check on force or $mail_check reached */
- if (force || (now - last1 >= BuffyTimeout)) {
- tmp->new = 0;
- tmp->msg_unread = 0;
+ if (force != 0 || (now - last1 >= BuffyTimeout)) {
if ((tmp->new = mh_buffy (tmp->path)) > 0)
BuffyCount++;
if (SidebarWidth > 0 && option (OPTMBOXPANE))
if ((tmp->new = mh_buffy (tmp->path)) > 0)
BuffyCount++;
if (SidebarWidth > 0 && option (OPTMBOXPANE))
@@
-523,7
+479,9
@@
int mutt_buffy_check (int force)
struct dirent *de;
if ((dp = opendir (path)) == NULL)
break;
struct dirent *de;
if ((dp = opendir (path)) == NULL)
break;
- tmp->msgcount = 0;
+ tmp->new = 0;
+ tmp->msgcount = 0;
+ tmp->msg_unread = 0;
while ((de = readdir (dp)))
{
if (mh_valid_message (de->d_name))
while ((de = readdir (dp)))
{
if (mh_valid_message (de->d_name))
@@
-535,33
+493,27
@@
int mutt_buffy_check (int force)
}
closedir (dp);
}
}
closedir (dp);
}
- tmp->has_new = tmp->new > 0;
- } else if (tmp->new > 0) {
+ } else if (tmp->new > 0)
/* keep current stats if !force and !$mail_check reached */
BuffyCount++;
/* keep current stats if !force and !$mail_check reached */
BuffyCount++;
- tmp->has_new = 1;
- }
break;
#ifdef USE_IMAP
case M_IMAP:
/* only check on force or $imap_mail_check reached */
break;
#ifdef USE_IMAP
case M_IMAP:
/* only check on force or $imap_mail_check reached */
- if (force || (now - last2 >= ImapBuffyTimeout)) {
- tmp->new = 0;
- tmp->msg_unread = 0;
+ if (force != 0 || (now - last2 >= ImapBuffyTimeout)) {
tmp->msgcount = imap_mailbox_check (tmp->path, 0);
if ((tmp->new = imap_mailbox_check (tmp->path, 1)) > 0) {
BuffyCount++;
tmp->msgcount = imap_mailbox_check (tmp->path, 0);
if ((tmp->new = imap_mailbox_check (tmp->path, 1)) > 0) {
BuffyCount++;
- tmp->has_new = tmp->new > 0;
tmp->msg_unread = tmp->new; /* for sidebar; wtf? */
}
tmp->msg_unread = tmp->new; /* for sidebar; wtf? */
}
- else
+ else
{
tmp->new = 0;
tmp->new = 0;
- } else if (tmp->new > 0) {
+ tmp->msg_unread = 0;
+ }
+ } else if (tmp->new > 0)
/* keep current stats if !force and !$imap_mail_check reached */
BuffyCount++;
/* keep current stats if !force and !$imap_mail_check reached */
BuffyCount++;
- tmp->has_new = 1;
- }
break;
#endif
break;
#endif
@@
-587,7
+539,7
@@
int mutt_buffy_check (int force)
BuffyNotify++;
tmp->has_new = tmp->new > 0;
}
BuffyNotify++;
tmp->has_new = tmp->new > 0;
}
- if (BuffyCount > 0)
+ if (BuffyCount > 0
&& force != 2
)
draw_sidebar (CurrentMenu);
return (BuffyCount);
}
draw_sidebar (CurrentMenu);
return (BuffyCount);
}