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

lazy "for" untested #529

Open
JJ opened this issue Apr 15, 2019 · 3 comments
Open

lazy "for" untested #529

JJ opened this issue Apr 15, 2019 · 3 comments

Comments

@JJ
Copy link
Contributor

JJ commented Apr 15, 2019

If I understand correctly this:

https://github.com/rakudo/rakudo/blob/f1c80fdd9310c14208e511c61f04f264b0f4dae7/src/Perl6/Grammar.nqp#L1403-L1409

lazy only works correctly in front of for statements, but that particular syntax does not seem to be tested.
@jnthn
Copy link
Contributor

jnthn commented Apr 15, 2019

It works in front of anything (see the | <blorst> below), it's just that the for statement case is special-cased.

@JJ
Copy link
Contributor Author

JJ commented Apr 15, 2019

Ah, OK. Then it should be tested anyway, right?

@jnthn
Copy link
Contributor

jnthn commented Apr 15, 2019

Well, since the lazy for .. case has its own special code path there, it'd seem especially deserving of a test. :-)

JJ added a commit to Raku/doc that referenced this issue Apr 15, 2019
@JJ
Fixes description of lazy statement prefix, refs #534 #2034
4ef6886 
Thanks for @jnthn clarification in Raku/roast#529
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JJ @jnthn