- Replaced embedded contexts with passing to functions
- Made as much as possible private
- Made Menu.Populate concurrent

@@ -0,0 +15,4 @@
+// example in the errgroup docs struck me as bound to deadlock, due to unbuffered
+// channels and calling Wait before consuming the channel. And I was right.
+func Test_MenuPopulateWithErrGroupConcept(t *testing.T) {
+	t.Skip("Manual test, disabled")

@@ -67,0 +68,4 @@
+// the button order gets jumbled after the first paging operation.
+// After a paging operation in EX, one can observe the buttons being out of order directly after a
+// click, but they sort and rearrange themselves back in order quickly.
+// I have no clue as to why this is.

@@ -68,6 +70,41 @@ func (bot *Client) CloseDB() error {
 	return bot.db.Close()
 }
 
+func (bot *Client) Go(f func()) {

@@ -0,0 +41,4 @@
+	}
+}
+
+// NewGeneratorFromIterator is just like NewGeneratorFromList, exceppt it takes an iterator

@@ -0,0 +107,4 @@
+}
+
+// NewWaitGroupWorker is nearly identical to NewDefaultWorker, except it uses the passed
+// WaitGroup to spawn goroutines, so that the caller may call wg.Wait() in the end.