UnexpectedCharacterException is thrown when a long table is being parsed

[whc2001]

When I tried to parse a very long table to LTable, e.g.

data = {
		[1] = {
			[1] = 55,
			[2] = 123,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试测试测试测试测试",
			[6] = "",
		},
		[2] = {
			[1] = 8,
			[2] = 12345,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试测试测试测试测试",
			[6] = "",
		},
		[3] = {
			[1] = 8,
			[2] = 65423,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "",
			[6] = "",
		},
		[4] = {
			[1] = 9,
			[2] = 84535,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "",
			[6] = "",
		},
		[5] = {
			[1] = 1,
			[2] = 16540,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试",
			[6] = "测试",
		},
		[6] = {
			[1] = 5,
			[2] = 23454,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试",
			[6] = "测试",
		},
		[7] = {
			[1] = 4,
			[2] = 24653,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试测试",
			[6] = "测试",
		},
		[8] = {
			[1] = 2,
			[2] = 55232,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试",
			[6] = "测试",
		},
		[9] = {
			[1] = 3,
			[2] = 45678,
			[3] = 0,
			[4] = "测试测试测试",
			[5] = "测试测试测试测试",
			[6] = "测试",
		},
}

After reading ~1000 characters, an UnexpectedCharacterException will be thrown. If I read less than that it works correctly. Seems like something is wrong about the buffer mechanism.

[whc2001]

Seems like the problem resides here.

Luaon.NET/Luaon.NET/LuaTableTextReader.cs

Lines 226 to 234 in 1623f56

	if (bufferLength + charsNeeded > buffer.Length)
	{
	// Shift unread chars to the left; reduce buffer length.
	Array.Copy(buffer, bufferPos, buffer, 0, bufferLength - bufferPos);
	bufferLength = bufferLength - bufferPos;
	bufferPos = 0;
	}
	var charsRead = Reader.ReadBlock(buffer, bufferPos, charsNeeded);
	bufferLength += charsRead;

When the buffer is about to exhaust, copy unread bytes to beginning, then append newly read bytes AFTER it. However on line 231, bufferPos is already set to zero, so on line 233 the further reading progress will NOT append the new bytes to the tail of unread byte, but rather the BEGINNING of the buffer, which will cause the unread bytes just moved to the beginning to be overwritten. Changing bufferPos to bufferLength on line 223 seems to solve the issue, since the appending will always be after the current buffer length.

[CXuesong]

Released in v0.2.3.

[whc2001]

(Moved to #3)