Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

attempt stops propagation of break #2417

Closed
maximvl opened this issue Jan 17, 2017 · 1 comment
Closed

attempt stops propagation of break #2417

maximvl opened this issue Jan 17, 2017 · 1 comment
Assignees
@dockimbel
Labels
status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. type.review Ticket describes a possible improvement.
Milestone
0.6.2

Comments

@maximvl
Copy link

maximvl commented Jan 17, 2017 •
edited 

red>> forever [ print "1" break ]
1
red>> forever [ attempt [ print "1" break ] ]
1
*** Throw Error: no loop to break
*** Where: break

In second case break is expected to break a loop, not to raise an error.
@dockimbel dockimbel added the type.review Ticket describes a possible improvement. label Feb 13, 2017
@dockimbel dockimbel self-assigned this Feb 13, 2017
@dockimbel dockimbel added the status.impending Ticket is under processing or will be processed very soon. label Feb 13, 2017
dockimbel added a commit that referenced this issue Feb 13, 2017
@dockimbel
FIX: issue #2417 (attempt stops propagation of break).
cef0d1b
@dockimbel
Copy link
Member

BREAK and CONTINUE exceptions have been relaxed to allow crossing function borders up to the first wrapping loop.

@dockimbel dockimbel added status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. and removed status.impending Ticket is under processing or will be processed very soon. labels Feb 13, 2017
@dockimbel dockimbel added this to the 0.6.2 milestone Feb 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dockimbel dockimbel

Labels
status.built A change in codebase has been done to address the ticket. status.tested The change in code has been manually tested and verified to fix the issue. type.review Ticket describes a possible improvement.
Projects
None yet
Milestone
0.6.2
Development

No branches or pull requests
2 participants
@dockimbel @maximvl