dovecot-1.0: Removed pointless maildir_uidlist_sync_next_pre() c...

dovecot at dovecot.org dovecot at dovecot.org
Sat Dec 1 10:39:20 EET 2007


details:   http://hg.dovecot.org/dovecot-1.0/rev/9f6b9a7e474c
changeset: 5474:9f6b9a7e474c
user:      Timo Sirainen <tss at iki.fi>
date:      Sat Dec 01 10:29:12 2007 +0200
description:
Removed pointless maildir_uidlist_sync_next_pre() calling.

diffstat:

3 files changed, 38 deletions(-)
src/lib-storage/index/maildir/maildir-sync.c    |   14 --------------
src/lib-storage/index/maildir/maildir-uidlist.c |   21 ---------------------
src/lib-storage/index/maildir/maildir-uidlist.h |    3 ---

diffs (68 lines):

diff -r f8ba40601922 -r 9f6b9a7e474c src/lib-storage/index/maildir/maildir-sync.c
--- a/src/lib-storage/index/maildir/maildir-sync.c	Sat Dec 01 09:39:20 2007 +0200
+++ b/src/lib-storage/index/maildir/maildir-sync.c	Sat Dec 01 10:29:12 2007 +0200
@@ -806,20 +806,6 @@ static int maildir_scan_dir(struct maild
 	while ((dp = readdir(dirp)) != NULL) {
 		if (dp->d_name[0] == '.')
 			continue;
-
-		ret = maildir_uidlist_sync_next_pre(ctx->uidlist_sync_ctx,
-						    dp->d_name);
-		if (ret == 0) {
-			/* new file and we couldn't lock uidlist, check this
-			   later in next sync. */
-			if (new_dir)
-				ctx->mbox->last_new_mtime = 0;
-			else
-				ctx->mbox->dirty_cur_time = ioloop_time;
-			continue;
-		}
-		if (ret < 0)
-			break;
 
 		check_touch = FALSE;
 		flags = 0;
diff -r f8ba40601922 -r 9f6b9a7e474c src/lib-storage/index/maildir/maildir-uidlist.c
--- a/src/lib-storage/index/maildir/maildir-uidlist.c	Sat Dec 01 09:39:20 2007 +0200
+++ b/src/lib-storage/index/maildir/maildir-uidlist.c	Sat Dec 01 10:29:12 2007 +0200
@@ -883,27 +883,6 @@ maildir_uidlist_sync_next_partial(struct
 	return 1;
 }
 
-int maildir_uidlist_sync_next_pre(struct maildir_uidlist_sync_ctx *ctx,
-				  const char *filename)
-{
-	if (!UIDLIST_IS_LOCKED(ctx->uidlist) &&
-	    hash_lookup(ctx->uidlist->files, filename) == NULL &&
-	    (ctx->partial || hash_lookup(ctx->files, filename) == NULL)) {
-		if (!ctx->uidlist->initial_read) {
-			/* first time reading the uidlist */
-			if (maildir_uidlist_update(ctx->uidlist) < 0) {
-				ctx->failed = TRUE;
-				return -1;
-			}
-			return maildir_uidlist_sync_next_pre(ctx, filename);
-		}
-
-		return 0;
-	}
-
-	return 1;
-}
-
 int maildir_uidlist_sync_next(struct maildir_uidlist_sync_ctx *ctx,
 			      const char *filename,
 			      enum maildir_uidlist_rec_flag flags)
diff -r f8ba40601922 -r 9f6b9a7e474c src/lib-storage/index/maildir/maildir-uidlist.h
--- a/src/lib-storage/index/maildir/maildir-uidlist.h	Sat Dec 01 09:39:20 2007 +0200
+++ b/src/lib-storage/index/maildir/maildir-uidlist.h	Sat Dec 01 10:29:12 2007 +0200
@@ -50,9 +50,6 @@ int maildir_uidlist_sync_init(struct mai
 int maildir_uidlist_sync_init(struct maildir_uidlist *uidlist,
 			      enum maildir_uidlist_sync_flags sync_flags,
 			      struct maildir_uidlist_sync_ctx **sync_ctx_r);
-/* Returns 1 = ok, -1 = error, 0 = new file and dovecot-uidlist is locked */
-int maildir_uidlist_sync_next_pre(struct maildir_uidlist_sync_ctx *ctx,
-				  const char *filename);
 int maildir_uidlist_sync_next(struct maildir_uidlist_sync_ctx *ctx,
 			      const char *filename,
 			      enum maildir_uidlist_rec_flag flags);


More information about the dovecot-cvs mailing list