jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[bugfix] fix delete in strutils and sequtils (#11535)

Browse source
This commit is contained in:
Miran 2019-06-20 21:04:37 +02:00 • committed by Andreas Rumpf
commit 23e95dc629
2 changed files with 5 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/pure/collections/sequtils.nim
View file

@ -432,7 +432,7 @@ proc delete*[T](s: var seq[T]; first, last: Natural) =
assert outcome == dest assert outcome == dest
var i = first var i = first
var j = last+1 var j = min(len(s), last+1)
var newLen = len(s)-j+i var newLen = len(s)-j+i
while i < newLen: while i < newLen:
s[i].shallowCopy(s[j]) s[i].shallowCopy(s[j])

5
lib/pure/strutils.nim
View file

@ -1494,8 +1494,11 @@ proc delete*(s: var string, first, last: int) {.noSideEffect,
a.delete(1, 6) a.delete(1, 6)
doAssert a == "ara" doAssert a == "ara"
a.delete(2, 999)
doAssert a == "ar"
var i = first var i = first
var j = last+1 var j = min(len(s), last+1)
var newLen = len(s)-j+i var newLen = len(s)-j+i
while i < newLen: while i < newLen:
s[i] = s[j] s[i] = s[j]