From 1b7790553e04816c3dd677848b2584ffd6785220 Mon Sep 17 00:00:00 2001 From: Aditya Mahajan Date: Fri, 14 Jun 2019 17:11:17 -0400 Subject: Ensure that flags are set properly The current implementation has three classes of flags: - readFlag - delFlag - flaggedFlag The logic to check for them should be in parallel if branches rather than in sequential if-else ladder. --- lib/indexformat.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'lib') diff --git a/lib/indexformat.go b/lib/indexformat.go index 8a329db..9e7a805 100644 --- a/lib/indexformat.go +++ b/lib/indexformat.go @@ -177,10 +177,12 @@ func ParseIndexFormat(conf *config.AercConfig, number int, readFlag = "N" // message is new } else if flag == imap.AnsweredFlag { readFlag = "r" // message has been replied to - } else if flag == imap.DeletedFlag { + } + if flag == imap.DeletedFlag { delFlag = "D" // TODO: check if attachments - } else if flag == imap.FlaggedFlag { + } + if flag == imap.FlaggedFlag { flaggedFlag = "!" } // TODO: check gpg stuff -- cgit v1.2.3