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

Fix unused warning for $ for empty tuple/objects (#13991)

Browse source
This commit is contained in:
hlaaftana 2020-04-18 18:41:03 +03:00 • committed by GitHub
commit f46803b225
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
lib/system/dollars.nim
View file

@ -75,7 +75,8 @@ proc `$`*[T: tuple|object](x: T): string =
## $(a: 23, b: 45) == "(a: 23, b: 45)" ## $(a: 23, b: 45) == "(a: 23, b: 45)"
## $() == "()" ## $() == "()"
result = "(" result = "("
var firstElement = true # when x is empty, this gives an unused warning
var firstElement {.used.} = true
const isNamed = T is object or isNamedTuple(T) const isNamed = T is object or isNamedTuple(T)
when not isNamed: when not isNamed:
var count = 0 var count = 0