From 4aaada18e9a32ca06cedc96da4ea30c8f3669b73 Mon Sep 17 00:00:00 2001 From: Oswald Buddenhagen Date: Sat, 11 Jul 2020 16:21:20 +0200 Subject: [PATCH] remove redundant condition the 'pending' and 'skipped' sync record states are mutually exclusive with having a complementary message, so there is no point in testing it explicitly. amends bd5fb6ff. --- src/sync.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/sync.c b/src/sync.c index 1eb5839..7ffd180 100644 --- a/src/sync.c +++ b/src/sync.c @@ -1588,8 +1588,7 @@ box_loaded( int sts, message_t *msgs, int total_msgs, int recent_msgs, void *aux // in case of interruption - in particular skipping big messages would otherwise // up the limit too early. srec = tmsg->srec; - if (srec ? !srec->uid[t] && - (((srec->status & S_PENDING) && (svars->chan->ops[t] & OP_NEW)) || + if (srec ? (((srec->status & S_PENDING) && (svars->chan->ops[t] & OP_NEW)) || ((srec->status & S_SKIPPED) && (svars->chan->ops[t] & OP_RENEW))) : svars->newmaxuid[1-t] < tmsg->uid && (svars->chan->ops[t] & OP_NEW)) { debug( "new message %u on %s\n", tmsg->uid, str_fn[1-t] );