From a81467dda9d577be706a1875451a3b031840639d Mon Sep 17 00:00:00 2001
From: Drew DeVault <sir@cmpwn.com>
Date: Sun, 2 Jun 2019 13:20:02 -0400
Subject: Remove worker callbacks when Done is received

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

diff --git a/worker/types/messages.go b/worker/types/messages.go
index 0169e05..4e46cbf 100644
--- 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
index 38140e0..85c19f1 100644
--- 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
-- 
cgit v1.2.3