From 564601afe7ce103a13cedb2025e52bb825c7ee25 Mon Sep 17 00:00:00 2001 From: "Tristan B. Velloza Kildaire" Date: Fri, 30 Jun 2023 12:33:51 +0200 Subject: [PATCH] Client - Now `doThreadCleanup()` will set the `receiver`, `sender` and `engine` to null such that no referenecs to it remain such that they can be gc'd --- source/birchwood/client/client.d | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/source/birchwood/client/client.d b/source/birchwood/client/client.d index 210a1ef..9f1cfb9 100644 --- a/source/birchwood/client/client.d +++ b/source/birchwood/client/client.d @@ -1024,21 +1024,17 @@ public class Client : Thread /* Stop the receive queue manager and wait for it to stop */ receiver.end(); logger.log("doThreadCleanup() recvQueue manager stopped"); - - // TODO: Set to null - + receiver = null; /* Stop the send queue manager and wait for it to stop */ sender.end(); logger.log("doThreadCleanup() sendQueue manager stopped"); - - // TODO: Set to null + sender = null; /* TODO: Stop eventy (FIXME: I don't know if this is implemented in Eventy yet, do this!) */ engine.shutdown(); logger.log("doThreadCleanup() eventy stopped"); - - // TODO: Set to null + engine = null; logger.log("doThreadCleanup() end"); }