Fix inlining detection (Fixes #99)

Author: xoofx

src/Tomlyn.Tests/SerializationTests.cs

@@ -26,4 +26,22 @@ public void TestCrlfInMultilineString()
 
         Assert.AreEqual("property = '''string\r\nwith\r\nnewlines'''" + Environment.NewLine, Toml.FromModel(model));
     }
+
+    [Test]
+    public void TestArrayWithPrimitives()
+    {
+        var model = new TomlTable()
+        {
+            ["mixed-array"] = new TomlArray()
+            {
+                new TomlTable() { ["a"] = 1 },
+                2,  // If instead the second or final item in the array was the table, it works as expected.
+                3
+            }
+        };
+
+        var result = Toml.FromModel(model).ReplaceLineEndings("\n").Trim();
+        Console.WriteLine(result);
+        Assert.AreEqual("mixed-array = [{a = 1}, 2, 3]", result);
+    }
 }

src/Tomlyn/Model/ModelToTomlTransform.cs

@@ -437,24 +437,30 @@ private bool IsRequiringInline(ListDynamicAccessor accessor, object value, int p
         // This is only working for default TomlTableArray model
         if (value is TomlTableArray) return false;
 
+        bool inlining = false;
+
         foreach (var element in accessor.GetElements(value))
         {
             if (element is null) continue; // TODO: should this log an error?
             var elementAccessor = _context.GetAccessor(element.GetType());
 
-            if (elementAccessor is PrimitiveDynamicAccessor) return true;
+            if (elementAccessor is PrimitiveDynamicAccessor)
+            {
+                return true;
+            }
 
             if (elementAccessor is ListDynamicAccessor listDynamicAccessor)
             {
-                return IsRequiringInline(listDynamicAccessor, element, parentConsecutiveList + 1);
-
+                inlining = IsRequiringInline(listDynamicAccessor, element, parentConsecutiveList + 1);
             }
             else if (elementAccessor is ObjectDynamicAccessor objAccessor)
             {
                 // Case of an array-of-array of table
                 if (parentConsecutiveList > 1) return true;
-                return IsRequiringInline(objAccessor, element);
+                inlining = IsRequiringInline(objAccessor, element);
             }
+
+            if (inlining) return true;
         }
 
         // Specially for empty list