From 5076169117796bb4cbd85b8a1c72891cc4810c09 Mon Sep 17 00:00:00 2001
From: "Tristan B. Velloza Kildaire" <deavmi@redxen.eu>
Date: Mon, 3 Jul 2023 18:05:29 +0200
Subject: [PATCH] Client

- Removed TODOs from `sendMessage(Message)`
- Added TODOs to to `channelMessage(string, stringp[])`, `channelMessage(string, string)`, `directMessage(string, string[])` and `directMessage(string, string)`
---
 source/birchwood/client/client.d | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/source/birchwood/client/client.d b/source/birchwood/client/client.d
index 89e4568..b33a257 100644
--- a/source/birchwood/client/client.d
+++ b/source/birchwood/client/client.d
@@ -475,6 +475,8 @@ public class Client : Thread
      */
     public void directMessage(string message, string[] recipients)
     {
+        // TODO: Chunked sends when over limit of `message`
+        
         /* Single recipient */
         if(recipients.length == 1)
         {
@@ -550,6 +552,8 @@ public class Client : Thread
      */
     public void directMessage(string message, string recipient)
     {
+        // TODO: Chunked sends when over limit of `message`
+
         /* Ensure the message and recipient are valid text */
         if(isValidText(message) && isValidText(recipient))
         {
@@ -582,6 +586,8 @@ public class Client : Thread
      */
     public void channelMessage(string message, string[] channels)
     {
+        // TODO: Chunked sends when over limit of `message`
+
         /* If single channel */
         if(channels.length == 1)
         {
@@ -657,6 +663,8 @@ public class Client : Thread
      */
     public void channelMessage(string message, string channel)
     {
+        // TODO: Chunked sends when over limit of `message`
+
         //TODO: Add check on recipient
         //TODO: Add emptiness check
         if(isValidText(message) && isValidText(channel))
@@ -954,17 +962,9 @@ public class Client : Thread
      */
     private void sendMessage(Message message)
     {
-        // TODO: Do message splits here
-        
         /* Encode the message */
         ubyte[] encodedMessage = encodeMessage(message.encode());
 
-        // TODO: Decide on the chunks required
-        ulong chunksRequired = encodedMessage.length/512;
-        // FIXME: I mean should this maybe be something that channelMessage
-        // ... does as I don't feel nor think we can issue several messages in general
-
-
         /* If the message is 512 bytes or less then send */
         if(encodedMessage.length <= 512)
         {