BRNSystems/minetest
1
0
Fork 0
forked from Mirrorlandia_minetest/minetest
Code Pull Requests 1 Activity

Fix string.split returning an empty table if string starts with sepearator (#7827)

Browse Source 
Calling string.split(":A:B:C:D", ":") returns an empty array.
This is due to first empty string not making repeat loop decreasing max_split which has a 0 value when reaching until.
Changing max_splits default value from -1 to -2 fixes that issue.
This commit is contained in:
Pierre-Yves Rollo 2018-11-01 21:07:01 +01:00 committed by SmallJoker
parent 4d4bfb1251
commit 0e306c0842
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
builtin/common/misc_helpers.lua
View File

@ -166,7 +166,7 @@ end
-------------------------------------------------------------------------------- --------------------------------------------------------------------------------
function string.split(str, delim, include_empty, max_splits, sep_is_pattern) function string.split(str, delim, include_empty, max_splits, sep_is_pattern)
delim = delim or "," delim = delim or ","
max_splits = max_splits or -1 max_splits = max_splits or -2
local items = {} local items = {}
local pos, len = 1, #str local pos, len = 1, #str
local plain = not sep_is_pattern local plain = not sep_is_pattern