diff options
| author | Reto Brunner <reto@labrat.space> | 2020-05-09 11:50:31 +0200 |
|---|---|---|
| committer | Drew DeVault <sir@cmpwn.com> | 2020-05-11 09:47:34 -0400 |
| commit | ea2646fc039a572491ba8cc8e2879b7bf61c25dd (patch) | |
| tree | 3608b8064e4f67f1f53e177ddad2e5f5ae9a49d5 /widgets/msgviewer.go | |
| parent | 381c1fc05f6de95accbb520769d0cc9196955cf4 (diff) | |
| download | aerc-ea2646fc039a572491ba8cc8e2879b7bf61c25dd.tar.gz | |
Change MarkedMessages to return uids
Especially if one tries to interact with all marked messages there could be the case that not all headers are fetched yet, hence the messageInfo is still nil. This segfaults a lot of commands which in principle only need the uid to complete. If we switch to uids, this issue can be alleviated for those commands.
Diffstat (limited to 'widgets/msgviewer.go')
| -rw-r--r-- | widgets/msgviewer.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/widgets/msgviewer.go b/widgets/msgviewer.go index e192ae6..0cfabd7 100644 --- a/widgets/msgviewer.go +++ b/widgets/msgviewer.go @@ -262,9 +262,9 @@ func (mv *MessageViewer) SelectedMessage() (*models.MessageInfo, error) { return mv.msg.MessageInfo(), nil } -func (mv *MessageViewer) MarkedMessages() ([]*models.MessageInfo, error) { +func (mv *MessageViewer) MarkedMessages() ([]uint32, error) { store := mv.Store() - return msgInfoFromUids(store, store.Marked()) + return store.Marked(), nil } func (mv *MessageViewer) ToggleHeaders() { |