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

Sets need copying in JS (#11392)

Browse source 
* Sets need copying in JS
* Fixes #11353
* Add test for #11353
This commit is contained in:
hlaaftana 2019-10-02 22:01:59 +03:00 • committed by Andreas Rumpf
commit ba43156363
2 changed files with 15 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
compiler/jsgen.nim
View file

@ -975,7 +975,7 @@ proc countJsParams(typ: PType): int =
const const
nodeKindsNeedNoCopy = {nkCharLit..nkInt64Lit, nkStrLit..nkTripleStrLit, nodeKindsNeedNoCopy = {nkCharLit..nkInt64Lit, nkStrLit..nkTripleStrLit,
nkFloatLit..nkFloat64Lit, nkCurly, nkPar, nkStringToCString, nkFloatLit..nkFloat64Lit, nkPar, nkStringToCString,
nkObjConstr, nkTupleConstr, nkBracket, nkObjConstr, nkTupleConstr, nkBracket,
nkCStringToString, nkCall, nkPrefix, nkPostfix, nkInfix, nkCStringToString, nkCall, nkPrefix, nkPostfix, nkInfix,
nkCommand, nkHiddenCallConv, nkCallStrLit} nkCommand, nkHiddenCallConv, nkCallStrLit}

14
tests/js/t11353.nim Normal file
View file

@ -0,0 +1,14 @@
discard """
output: '''
{}
{}
'''
"""
proc foo() =
var bar: set[int16] = {}
echo bar
bar.incl(1)
foo()
foo()