Messages

- Slightly improved trailer detection
- Added some TODOs and notes

Client

- Log the key-value parameters and trailer

source/birchwood/client/client.d

@@ -140,6 +140,9 @@ public class Client : Thread
         if(commandReply.getReplyType() == ReplyType.RPL_BOUNCE)
         {
             logger.log("Take a look:\n\n"~commandReply.getParams());
+
+            logger.log("And here is key-value pairs: ", commandReply.getKVPairs());
+            logger.log("And here is everything: ", commandReply.getTrailing());
         }
     }
 

source/birchwood/protocol/messages.d

@@ -293,6 +293,12 @@ public final class Message
     private string ppTrailing;
     private string[string] ppKVPairs;
 
+    /** 
+     * NOTE: This needs more work with trailing support
+     * we must make sure we only look for lastInex of `:`
+     * where it is first cyaracter after space but NOT within
+     * an active parameter
+     */
     private void parameterParse()
     {
         /* Only parse if there are params */
@@ -308,7 +314,10 @@ public final class Message
             string trailing;
 
             /* Find the first (and should be only) : (if any) */
-            long trailingIdx = indexOf(params, ":");
+            // TODO: Maybe i misunderstoof it as it appears in
+            // some key-value pairs as the value
+            // ... therefore let's just look for the last one
+            long trailingIdx = lastIndexOf(params, ":");
 
             /* If there is trailing */
             if(trailingIdx > -1)
@@ -318,6 +327,8 @@ public final class Message
 
                 /* Save the trailing text */
                 trailing = params[trailingIdx+1..params.length];
+
+                logger.debug_("Look at this trailing '"~trailing~"'");
             }
             /* If there is no trailing */
             else
@@ -326,6 +337,8 @@ public final class Message
                 kvPairs = params;
             }
 
+            // TODO: strip whitespace on either side of `kvPairs`
+
             /* Generate the key-value pairs */
             string[] pairs = split(kvPairs, " ");
             logger.debug_("Pairs: ", pairs);