diff options
| author | Tim Culverhouse <tim@timculverhouse.com> | 2022-08-16 12:25:48 -0500 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-08-22 15:46:49 +0200 |
| commit | a0a5ba1538d8b8108edbf737173cb0cee54caee5 (patch) | |
| tree | 7b19559f8e4991af72359dbd5761c09d92ac7408 /worker/imap/seqmap_test.go | |
| parent | d7e6dc3649eadc27bdfdac77785b6bbf77d30b79 (diff) | |
| download | aerc-a0a5ba1538d8b8108edbf737173cb0cee54caee5.tar.gz | |
imap: create copy of uids to retain sort order
Commit fdfec2c07a8d seqmap: refactor seqmap to use slice instead of map
introduced a regression to imap sorting (if supported). The slice passed
to seqmap was being sorted in place by UID, thus breaking any sort order
sent by the server.
Create a copy of the slice to retain the sort order reported to the UI
while also keeping a correct map for seqnum -> UID in the worker.
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'worker/imap/seqmap_test.go')
| -rw-r--r-- | worker/imap/seqmap_test.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/worker/imap/seqmap_test.go b/worker/imap/seqmap_test.go index 42c06f81..5d6cf796 100644 --- a/worker/imap/seqmap_test.go +++ b/worker/imap/seqmap_test.go @@ -22,8 +22,11 @@ func TestSeqMap(t *testing.T) { _, found = seqmap.Pop(0) assert.Equal(false, found) - seqmap.Initialize([]uint32{1337, 42, 1107}) + uids := []uint32{1337, 42, 1107} + seqmap.Initialize(uids) assert.Equal(3, seqmap.Size()) + // Original list should remain unsorted + assert.Equal([]uint32{1337, 42, 1107}, uids) _, found = seqmap.Pop(0) assert.Equal(false, found) |