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

Merge pull request #4980 from jlp765/tostring

Browse source 
Add nill value handling for seq/sets to system.CollectionToString()
This commit is contained in:
Andreas Rumpf 2016-11-01 09:45:37 +01:00 • committed by GitHub
commit 4884a56e38
2 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
lib/system.nim
View file

@ -2334,7 +2334,8 @@ proc collectionToString[T: set | seq](x: T, b, e: string): string =
var firstElement = true var firstElement = true
for value in items(x): for value in items(x):
if not firstElement: result.add(", ") if not firstElement: result.add(", ")
result.add($value) if value.isNil: result.add "nil"
else: result.add($value)
firstElement = false firstElement = false
result.add(e) result.add(e)

2
tests/stdlib/tnilecho.nim Normal file
View file

@ -0,0 +1,2 @@
var x = @["1", nil, "3"]
doAssert $x == "@[1, nil, 3]"