Allow for newline entity in MXP sequence

nickgammon · Feb 12, 2015 · 5ad24b1 · 5ad24b1
1 parent cba48ce
commit 5ad24b1
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/mxp/mxpEntities.cpp b/mxp/mxpEntities.cpp
@@ -47,7 +47,12 @@ void CMUSHclientDoc::MXP_collected_entity (void)
 CString strEntityContents = MXP_GetEntity (m_strMXPstring);
 
   if (!strEntityContents.IsEmpty ())
-    AddToLine (strEntityContents, 0);
+    {
+//  if the entity happens to be < & > etc. don't reprocess it
+    m_bMXP = false;
+    DisplayMsg (strEntityContents, strEntityContents.GetLength (), 0);
+    m_bMXP = true;
+    }
 
   } // end of CMUSHclientDoc::MXP_collected_entity
 
@@ -109,7 +114,7 @@ CString strEntityContents;
         iResult *= 10;
         iResult += strName [i] - '0';
         }
-    if (iResult != 9  && iResult != 10 )       // we will accept tabs and newlines ;)
+    if (iResult != 9  && iResult != 10  && iResult != 13 )       // we will accept tabs, newlines and carriage-returns ;)
       if (iResult < 32 ||   // don't allow nonprintable characters
           iResult > 255)   // don't allow characters more than 1 byte
           {

diff --git a/mxp/mxp_phases.cpp b/mxp/mxp_phases.cpp
@@ -82,8 +82,8 @@ void CMUSHclientDoc::Phase_MXP_ENTITY (const unsigned char c)
   switch (c)
     {
     case ';':
-        MXP_collected_entity (); // we have an entity now
         m_phase = NONE;
+        MXP_collected_entity (); // we have an entity now
         break;
 
         // shouldn't have a & inside a &