aerc

Working clone of aerc-mail.org
git clone git://git.danielmoch.com/aerc.git
Log | Files | Refs | README | LICENSE

commit a81467dda9d577be706a1875451a3b031840639d
parent 7d1770754ff432b0c6f153c066e4c749683b563f
Author: Drew DeVault <sir@cmpwn.com>
Date:   Sun,  2 Jun 2019 13:20:02 -0400

Remove worker callbacks when Done is received

Diffstat:
Mworker/types/messages.go | 2+-
Mworker/types/worker.go | 16++++++++++++----
2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/worker/types/messages.go b/worker/types/messages.go @@ -34,7 +34,7 @@ func (m Message) getId() int64 { return m.id } -func (m Message) setId(id int64) { +func (m *Message) setId(id int64) { m.id = id } diff --git a/worker/types/worker.go b/worker/types/worker.go @@ -72,13 +72,17 @@ func (worker *Worker) PostMessage(msg WorkerMessage, func (worker *Worker) ProcessMessage(msg WorkerMessage) WorkerMessage { if resp := msg.InResponseTo(); resp != nil { - worker.Logger.Printf("(ui)<= %T:%T\n", msg, resp) + worker.Logger.Printf("(ui)<= %T(%d):%T(%d)\n", + msg, msg.getId(), resp, resp.getId()) } else { - worker.Logger.Printf("(ui)<= %T\n", msg) + worker.Logger.Printf("(ui)<= %T(%d)\n", msg, msg.getId()) } if inResponseTo := msg.InResponseTo(); inResponseTo != nil { if f, ok := worker.actionCallbacks[inResponseTo.getId()]; ok { f(msg) + if _, ok := msg.(*Done); ok { + delete(worker.actionCallbacks, inResponseTo.getId()) + } } } return msg @@ -86,13 +90,17 @@ func (worker *Worker) ProcessMessage(msg WorkerMessage) WorkerMessage { func (worker *Worker) ProcessAction(msg WorkerMessage) WorkerMessage { if resp := msg.InResponseTo(); resp != nil { - worker.Logger.Printf("<-(ui) %T:%T\n", msg, resp) + worker.Logger.Printf("<-(ui) %T(%d):%T(%d)\n", + msg, msg.getId(), resp, resp.getId()) } else { - worker.Logger.Printf("<-(ui) %T\n", msg) + worker.Logger.Printf("<-(ui) %T(%d)\n", msg, msg.getId()) } if inResponseTo := msg.InResponseTo(); inResponseTo != nil { if f, ok := worker.messageCallbacks[inResponseTo.getId()]; ok { f(msg) + if _, ok := msg.(*Done); ok { + delete(worker.messageCallbacks, inResponseTo.getId()) + } } } return msg