Fixed bug eating \r eating line.

TradeWars Proxy Notes.md

@@ -7,7 +7,16 @@
 * define the functions (server_line, server_prompt [default to null/not set], client_input).  And defaults "to_client, to_server" -- but those won't ever change.  
 * factory of shared_ptr of "director" ? (from dispatcher) so it gets parent / auto saves on ctor.  Sounds like a complete mess.  :(
 * BUG:  Things like "Activate Proxy" -- needs a timer in order to be able to send " " every so often to keep the game alive.
-* 
+
+
+
+# BUG
+
+* none that this moment.  Get writing some!
+
+
+
+
 
 ## Things to Improve / Fix
 

dispatchers.h

@@ -4,7 +4,9 @@
 class Dispatch;
 #include "session.h"
 
-// possibly setup a timer to call Dispatch.
+// How does this call another?  
+// How does it return a result?
+
 
 class Dispatch {
 protected:

session.cpp

@@ -204,6 +204,9 @@ what was just added to server_prompt.
 
 What about \r, \b ?  Should that "reset" the server_prompt?
 
+\r should not, because it is followed by \n (eventually)
+and that completes my line.
+
  */
 void Session::process_lines(std::string &received) {
   // break server_prompt into lines and send/process one by one.
@@ -302,11 +305,15 @@ void Session::process_lines(std::string &received) {
       // BUGZ_LOG(error) << "show_client/leftovers:" << clean;
     }
 
+  // This is eating the entire string.  String is partial line
+  // portcim line, ending with '\r', this eats the line.
+  /*
   // check the server prompt here:
   if ((pos = server_prompt.rfind('\r')) != std::string::npos) {
     // server_prompt contains \r, remove it.
     server_prompt = server_prompt.substr(pos + 1);
   }
+  */
 
   while ((pos = server_prompt.find('\b')) != std::string::npos) {
     // backspace?  OK!  (unless)