Fix wrong amount of buffers created in constructor of StringDataFrameColumn

asmirnov82 · asmirnov82 · commit 251d955c4226 · 2023-10-02T20:13:43.000+03:00
diff --git a/src/Microsoft.Data.Analysis/StringDataFrameColumn.cs b/src/Microsoft.Data.Analysis/StringDataFrameColumn.cs
@@ -26,7 +26,7 @@ public partial class StringDataFrameColumn : DataFrameColumn, IEnumerable<string
 
         public StringDataFrameColumn(string name, long length = 0) : base(name, length, typeof(string))
         {
-            int numberOfBuffersRequired = Math.Max((int)(length / MaxCapacity), 1);
+            int numberOfBuffersRequired = (int)(length / MaxCapacity + 1);
             for (int i = 0; i < numberOfBuffersRequired; i++)
             {
                 long bufferLen = length - _stringBuffers.Count * MaxCapacity;

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ public partial class StringDataFrameColumn : DataFrameColumn, IEnumerable<string`
`26`	`26`
`27`	`27`	`public StringDataFrameColumn(string name, long length = 0) : base(name, length, typeof(string))`
`28`	`28`	`{`
`29`		`- int numberOfBuffersRequired = Math.Max((int)(length / MaxCapacity), 1);`
	`29`	`+ int numberOfBuffersRequired = (int)(length / MaxCapacity + 1);`
`30`	`30`	`for (int i = 0; i < numberOfBuffersRequired; i++)`
`31`	`31`	`{`
`32`	`32`	`long bufferLen = length - _stringBuffers.Count * MaxCapacity;`