Merge pull request #251 from sovietmagic/master

SvgPointCollection ToString method fix

Source/DataTypes/SvgPointCollection.cs

@@ -15,13 +15,22 @@ namespace Svg
     {
         public override string ToString()
         {
-            string ret = "";
-            foreach (var unit in this)
+            var builder = new StringBuilder();
+            for (var i = 0; i < Count; i += 2) 
             {
-                ret += unit.ToString() + " ";
+                if (i + 1 < Count) 
+                {
+                    if (i > 1) 
+                    {
+                        builder.Append(" ");
+                    }
+                    // we don't need unit type
+                    builder.Append(this[i].Value.ToString(CultureInfo.InvariantCulture));
+                    builder.Append(",");
+                    builder.Append(this[i + 1].Value.ToString(CultureInfo.InvariantCulture));
+                }    
             }
-
-            return ret;
+            return builder.ToString();
         }
     }
 

Tests/Svg.UnitTests/Svg.UnitTests.csproj

@@ -62,6 +62,7 @@
     <Compile Include="PrivateFontsTests.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="SmallEmbeddingImageTest.cs" />
+    <Compile Include="SvgPointCollectionTests.cs" />
     <Compile Include="SvgTestHelper.cs" />
   </ItemGroup>
   <ItemGroup>

Tests/Svg.UnitTests/SvgPointCollectionTests.cs

+using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+namespace Svg.UnitTests {
+
+    [TestClass]
+    public class SvgPointCollectionTests 
+    {
+
+        [TestMethod]
+        public void ToStringReturnsValidString() 
+        {
+            var collection = new SvgPointCollection 
+            {
+                new SvgUnit(1.6f), new SvgUnit(3.2f),
+                new SvgUnit(1.2f), new SvgUnit(5f)
+            };
+            Assert.AreEqual("1.6,3.2 1.2,5", collection.ToString());
+        }
+    }
+}