mantarimay/Aegisub
1
0
Fork 0
forked from mia/Aegisub
Code Pull requests Activity

Fix skipping trailing empty segments in re.split

Browse source
This commit is contained in:
Thomas Goyne 2014-09-20 08:13:49 -07:00
parent 9e6d5d7d9c
commit 57d66be785
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
automation/include/aegisub/re.moon
View file

@ -165,7 +165,7 @@ class RegEx
if not first or first > str\len() if not first or first > str\len()
ret = str\sub prev, str\len() ret = str\sub prev, str\len()
str = nil str = nil
return ret return if skip_empty and ret\len() == 0 then nil else ret
ret = str\sub prev, first - 1 ret = str\sub prev, first - 1
prev = last + 1 prev = last + 1

5
automation/tests/modules/re.moon
View file

@ -170,6 +170,11 @@ describe 'split', ->
assert.is.equal 'b', res[2] assert.is.equal 'b', res[2]
assert.is.equal 'c', res[3] assert.is.equal 'c', res[3]
it 'should return an empty table when given only empty segments', ->
res = re.split ',,,', ',', true
assert.is.not.nil res
assert.is.equal 0, #res
it 'should be able to split on word boundaries', -> it 'should be able to split on word boundaries', ->
res = re.split 'aa bb cc', '\\b', true res = re.split 'aa bb cc', '\\b', true
assert.is.not.nil res assert.is.not.nil res