Don't lose chars when parsing string into string[]

Author: mythz

src/ServiceStack.Text/Json/JsonTypeSerializer.cs

@@ -712,10 +712,10 @@ public bool EatItemSeperatorOrMapEndChar(ReadOnlySpan<char> value, ref int i)
 
             var success = value[i] == JsWriter.ItemSeperator || value[i] == JsWriter.MapEndChar;
 
-            i++;
-
             if (success)
             {
+                i++;
+
                 for (; i < value.Length; i++) { var c = value[i]; if (!JsonUtils.IsWhiteSpace(c)) break; } //Whitespace inline
             }
 

src/ServiceStack.Text/Jsv/JsvTypeSerializer.cs

@@ -389,11 +389,15 @@ public bool EatItemSeperatorOrMapEndChar(ReadOnlySpan<char> value, ref int i)
 
             var success = value[i] == JsWriter.ItemSeperator
                 || value[i] == JsWriter.MapEndChar;
-            i++;
+
+            if (success)
+            {
+                i++;
+            }
+            
             return success;
         }
 
-
         public void EatWhitespace(string value, ref int i) {}
 
         public void EatWhitespace(ReadOnlySpan<char> value, ref int i) { }

tests/ServiceStack.Text.Tests/Issues/InvalidParsingIssues.cs

@@ -0,0 +1,16 @@
+using NUnit.Framework;
+
+namespace ServiceStack.Text.Tests.Issues
+{
+    public class InvalidParsingIssues
+    {
+        [Test]
+        public void Parsing_string_into_string_array_splits_on_spaces()
+        {
+            var str = "string with a bunch of words";
+            var result = str.FromJson<string[]>();
+            
+            Assert.That(result, Is.EquivalentTo(str.Split(' ')));
+        }
+    }
+}