diff options
author | Moritz Poldrack <git@moritz.sh> | 2022-03-09 22:48:00 +0100 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2022-03-12 21:00:23 +0100 |
commit | 4bc43d2741fa4904e51fc5da71d15b804c556c43 (patch) | |
tree | 1747e08585ef0c98a13d3477369b4f159071a762 /commands/msg | |
parent | 65ae87a524ebbb573626afe951d6cd29bc8b24cd (diff) | |
download | aerc-4bc43d2741fa4904e51fc5da71d15b804c556c43.tar.gz |
all: fix minor issues found by staticcheck
Signed-off-by: Moritz Poldrack <git@moritz.sh>
Diffstat (limited to 'commands/msg')
-rw-r--r-- | commands/msg/delete.go | 15 | ||||
-rw-r--r-- | commands/msg/reply.go | 2 |
2 files changed, 8 insertions, 9 deletions
diff --git a/commands/msg/delete.go b/commands/msg/delete.go index 34eac72f..5eb622de 100644 --- a/commands/msg/delete.go +++ b/commands/msg/delete.go @@ -88,15 +88,14 @@ func findNextNonDeleted(deleted []uint32, store *lib.MessageStore) *models.Messa if !contains(deleted, selected.Uid) { return selected } - for { - store.Next() - next := store.Selected() - if next == selected || next == nil { - // the last message is in the deleted state or doesn't exist - return nil - } - return next + + store.Next() + next := store.Selected() + if next == selected || next == nil { + // the last message is in the deleted state or doesn't exist + return nil } + return next } func contains(uids []uint32, uid uint32) bool { diff --git a/commands/msg/reply.go b/commands/msg/reply.go index c3ab202c..30b09b62 100644 --- a/commands/msg/reply.go +++ b/commands/msg/reply.go @@ -122,7 +122,7 @@ func (reply) Execute(aerc *widgets.Aerc, args []string) error { } } if len(to) == 0 { - to = append(msg.Envelope.To) + to = msg.Envelope.To } } |