From cd002567e886d1692799747bfe529d72e024c16d Mon Sep 17 00:00:00 2001 From: Tim Culverhouse Date: Tue, 30 Aug 2022 18:24:59 -0500 Subject: [PATCH] msgstore: do not run threadbuilder unless in threaded view Commit 380cf13cff8e ("msgstore: run threadBuilder with no debounce on DirectoryContents msg") fixed an issue related to running the threadbuilder with a debounce. This fix causes the thread builder to run for any DirectoryContents message, even if the view is not threaded. Don't call runThreadBuilderNow unless in a threaded view. Fixes: 380cf13cff8e ("msgstore: run threadBuilder with no debounce on DirectoryContents msg") Signed-off-by: Tim Culverhouse Acked-by: Robin Jarry --- lib/msgstore.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/msgstore.go b/lib/msgstore.go index 5655f9e..33c2439 100644 --- a/lib/msgstore.go +++ b/lib/msgstore.go @@ -209,7 +209,9 @@ func (store *MessageStore) Update(msg types.WorkerMessage) { } store.Messages = newMap store.uids = msg.Uids - store.runThreadBuilderNow() + if store.threadedView { + store.runThreadBuilderNow() + } case *types.DirectoryThreaded: var uids []uint32 newMap := make(map[uint32]*models.MessageInfo)