Merge pull request #532 from synacktic/ezpz-string-fix

Remove the EZPZ truncation
mbbsemu · Jan 6, 2022 · 28580a3 · 28580a3
2 parents 6c2dce8 + c816ba3
commit 28580a3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/MBBSEmu.Tests/ExportedModules/Majorbbs/sprintf_Tests.cs b/MBBSEmu.Tests/ExportedModules/Majorbbs/sprintf_Tests.cs
@@ -40,6 +40,8 @@ public class sprintf_Tests : ExportedModuleTestBase
         [InlineData("%%%%", "%%", null)] //Escaped %
         [InlineData("%%%%%", "%%%", null)] //Escaped & Unescaped %
         [InlineData("%%%%% ", "%%% ", null)] //Escaped & Unescaped %
+        [InlineData("%-8s", "gold crowns", "gold crowns")] 
+        [InlineData("%-8s", "gold    ", "gold")] 
         public void sprintf_Test(string formatString, string expectedString, params object[] values)
         {
             Reset();

diff --git a/MBBSEmu/HostProcess/ExportedModules/ExportedModuleBase.cs b/MBBSEmu/HostProcess/ExportedModules/ExportedModuleBase.cs
@@ -644,10 +644,6 @@ private protected ReadOnlySpan<byte> FormatPrintf(ReadOnlySpan<byte> stringToPar
                                 msFormattedValue.Write(valueCache);
                             }
                         }
-
-                        //Need to truncate -- EZPZ
-                        if (msFormattedValue.Length > stringWidth)
-                            msFormattedValue.SetLength(stringWidth);
                     }
                     msOutput.Write(msFormattedValue.ToArray());
                     continue;