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

newruntime: fixes memory leak

Browse source
This commit is contained in:
Araq 2019-04-09 12:20:10 +02:00
commit 2430521c84
1 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

7
lib/core/strs.nim
View file

@ -163,11 +163,12 @@ proc mnewString(len: int): NimStringV2 {.compilerProc.} =
result = NimStringV2(len: len, p: p)
proc setLengthStrV2(s: var NimStringV2, newLen: int) {.compilerRtl.} =
if newLen > s.len or isLiteral(s):
if newLen == 0:
frees(s)
s.p = nil
elif newLen > s.len or isLiteral(s):
prepareAdd(s, newLen - s.len)
s.len = newLen
# this also only works because the destructor
# looks at s.p and not s.len
proc nimAsgnStrV2(a: var NimStringV2, b: NimStringV2) {.compilerRtl.} =
# self assignment is fine!