Why not use string.Split to split the grid length string ?

We need to parse the grid length string in the code below. But I think the easy way to do it is to use string.Split.

https://github.com/AvaloniaUI/Avalonia/blob/35b29d8f2367fcc62775bc4de7d5f8a3802d8c07/src/Avalonia.Controls/GridLength.cs#L212

And I try to use string.Split to replace this code and I found it can work and pass the unit test.

This is my code

      public static IEnumerable<GridLength> ParseLengths(string s, CultureInfo culture)
        {
            foreach (var temp in s.Split(new []{',',' '}, StringSplitOptions.RemoveEmptyEntries))
            {
                yield return Parse(temp, culture);
            }      
        }

[aelij]

@lindexi Definitely! We could probably eliminate all string allocations here, but I think we’ll need to wait for .NET Standard 2.1. Or provide alternate implementations for .NET Core and .NET Framework.

[aelij]

@Gillibald The real advantage from Span here would come from parsing APIs (e.g. Int32.TryParse(Span<char>)) and that’s currently available only in .NET Core 2.1.