diff options
| author | Tim Culverhouse <tim@timculverhouse.com> | 2022-12-09 08:03:35 -0600 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-12-14 11:24:49 +0100 |
| commit | 9c56417892736af67d5c67d34b6d59458e27fe27 (patch) | |
| tree | 26ade43533be82d00eb813046ac08e8249d20fc6 | |
| parent | d79458f464abccdb43e7d14753577698667ec81c (diff) | |
| download | aerc-9c56417892736af67d5c67d34b6d59458e27fe27.tar.gz aerc-9c56417892736af67d5c67d34b6d59458e27fe27.zip | |
findNextNonDeleted: set next to nil if last message
The introduction of the iterator means the "next" non-deleted message is
never nil, it will always be equal to the previous message (meaning
there is only one message left and it is the one we are deleting). In
this case, deliberately set next to nil so that the remove tab on delete
logic works properly.
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Acked-by: Moritz Poldrack <moritz@poldrack.dev>
| -rw-r--r-- | commands/msg/delete.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/commands/msg/delete.go b/commands/msg/delete.go index 6c786257..9a07e509 100644 --- a/commands/msg/delete.go +++ b/commands/msg/delete.go @@ -110,6 +110,9 @@ func findNextNonDeleted(deleted []uint32, store *lib.MessageStore) *models.Messa } } if next == nil || previous == next { + // If previous == next, this is the last + // message. Set next to nil either way + next = nil break } stepFn() |