blacklight/aerc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

widgets: remove redundant key check, delete is no-op if key is empty

Browse source 
Signed-off-by: Wagner Riffel <wgrriffel@gmail.com>
This commit is contained in:
Wagner Riffel 2019-09-03 16:34:07 -03:00 committed by Drew DeVault
parent bdb5e3e658
commit 6338194c4e
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/msgstore.go
View file

@ -105,11 +105,9 @@ func (store *MessageStore) FetchFull(uids []uint32, cb func(io.Reader)) {
switch msg.(type) { switch msg.(type) {
case *types.Error: case *types.Error:
for _, uid := range toFetch { for _, uid := range toFetch {
if _, ok := store.bodyCallbacks[uid]; ok {
delete(store.bodyCallbacks, uid) delete(store.bodyCallbacks, uid)
} }
} }
}
}) })
} }
} }
@ -209,10 +207,8 @@ func (store *MessageStore) Update(msg types.WorkerMessage) {
for _, uid := range msg.Uids { for _, uid := range msg.Uids {
toDelete[uid] = nil toDelete[uid] = nil
delete(store.Messages, uid) delete(store.Messages, uid)
if _, ok := store.Deleted[uid]; ok {
delete(store.Deleted, uid) delete(store.Deleted, uid)
} }
}
uids := make([]uint32, len(store.uids)-len(msg.Uids)) uids := make([]uint32, len(store.uids)-len(msg.Uids))
j := 0 j := 0
for _, uid := range store.uids { for _, uid := range store.uids {