tk/spectre.console
1
0
Fork 0
mirror of https://github.com/nsnail/spectre.console.git synced 2025-04-16 00:42:51 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix ArgumentOutOfRangeException when rendering a table

Browse Source 
When rendering a table with East Asia characters (take 2 English alphabets width) will throw ArgumentOutOfRangeException.
This commit is contained in:
Liam Sho 2021-08-12 21:50:31 +08:00 committed by Patrik Svensson
parent d96817dc9c
commit d306ad82d1
1 changed files with 28 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
src/Spectre.Console/Rendering/Segment.cs
View File

@ -3,6 +3,7 @@ using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;
using System.Text;
using Wcwidth;
namespace Spectre.Console.Rendering
{
@ -329,23 +330,10 @@ namespace Spectre.Console.Rendering
if (overflow == Overflow.Fold)
{
var totalLength = segment.Text.CellLength();
var lengthLeft = totalLength;
while (lengthLeft > 0)
var splitted = SplitSegment(segment.Text, maxWidth);
foreach (var str in splitted)
{
var index = totalLength - lengthLeft;
// How many characters should we take?
var take = Math.Min(maxWidth, totalLength - index);
if (take <= 0)
{
// This shouldn't really occur, but I don't like
// never ending loops if it does...
return new List<Segment>();
}
result.Add(new Segment(segment.Text.Substring(index, take), segment.Style));
lengthLeft -= take;
result.Add(new Segment(str, segment.Style));
}
}
else if (overflow == Overflow.Crop)
@ -567,5 +555,29 @@ namespace Spectre.Console.Rendering
return cells;
}
internal static List<string> SplitSegment(string text, int maxCellLength)
{
var list = new List<string>();
var length = 0;
var sb = new StringBuilder();
foreach (var ch in text)
{
if (length + UnicodeCalculator.GetWidth(ch) > maxCellLength)
{
list.Add(sb.ToString());
sb.Clear();
length = 0;
}
length += UnicodeCalculator.GetWidth(ch);
sb.Append(ch);
}
list.Add(sb.ToString());
return list;
}
}
}