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

thoughts about 'continue/break' bug

Browse source
This commit is contained in:
Araq 2012-05-10 23:26:38 +02:00
commit 8cf9b48ad4
2 changed files with 5 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
compiler/transf.nim
View file

@ -255,6 +255,10 @@ proc hasContinue(n: PNode): bool =
proc transformLoopBody(c: PTransf, n: PNode): PTransNode = proc transformLoopBody(c: PTransf, n: PNode): PTransNode =
# XXX BUG: What if it contains "continue" and "break"? "break" needs # XXX BUG: What if it contains "continue" and "break"? "break" needs
# an explicit label too, but not the same! # an explicit label too, but not the same!
# We fix this here by making every 'break' belong to its enclosing loop
# and changing all breaks that belong to a 'block' by annotating it with
# a label (if it hasn't one already).
if hasContinue(n): if hasContinue(n):
var labl = newSym(skLabel, nil, getCurrOwner(c)) var labl = newSym(skLabel, nil, getCurrOwner(c))
labl.name = getIdent(genPrefix & $labl.id) labl.name = getIdent(genPrefix & $labl.id)

1
todo.txt
View file

@ -1,6 +1,7 @@
version 0.9.0 version 0.9.0
============= =============
- fix transformLoopBody
- make templates hygienic by default - make templates hygienic by default
- ``bind`` for overloaded symbols does not work apparently - ``bind`` for overloaded symbols does not work apparently
- ``=`` should be overloadable; requires specialization for ``=`` - ``=`` should be overloadable; requires specialization for ``=``