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.

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

Rakudo recommends a different syntax for do...for loops and then that syntax doesn't work #2544

Closed
p6rt opened this issue Nov 1, 2011 · 8 comments
Labels

Comments

@p6rt
Copy link

p6rt commented Nov 1, 2011

Migrated from rt.perl.org#102652 (status was 'resolved')

Searchable as RT102652$

@p6rt
Copy link
Author

p6rt commented Nov 1, 2011

From @masak

<moritz> nom​: do { } for 1
<p6eval> nom 878004​: OUTPUT«===SORRY!===␤Unsupported use of do...for;
in Perl 6 please use repeat...for at line 1 [...]
<moritz> nom​: repeat { } for 1
<p6eval> nom 878004​: OUTPUT«===SORRY!===␤CHECK FAILED​:␤Undefined
routine '&repeat' called (line 1)␤»
<moritz> masak​: care to submit that one too?
* masak submits rakudobug
<moritz> it seems that 'repeat { } for ...' isn't supposed to work,
but then the error message shouldn't recommend it
<moritz> std​: repeat { } for 1
<p6eval> std be1f10e​: OUTPUT«===SORRY!===␤Undeclared routine​:␤
'repeat' used at line 1␤Check failed␤FAILED 00​:01 120m␤»
<moritz> std​: do { } for 1
<p6eval> std be1f10e​: OUTPUT«===SORRY!===␤Unsupported use of do...for;
in Perl 6 please use repeat...for [...]
<moritz> oh, and stdbug too
* moritz submits STDbug
<moritz> TimToady​: perl6/std#1

@p6rt
Copy link
Author

p6rt commented Nov 21, 2012

From @FROGGS

well, almost​:

<FROGGS> r​: do { } for 1
<p6eval> rakudo bf472b​: OUTPUT«===SORRY!===␤Unsupported use of do...for;
in Perl 6 please use repeat...for␤at /tmp/7Fy9NvtoTR​:1␤»

<FROGGS> r​: repeat { } for 1
<p6eval> rakudo bf472b​: OUTPUT«===SORRY!===␤"repeat" is missing its
"while" or "until" at line 2, near "for 1"␤»

@p6rt
Copy link
Author

p6rt commented Nov 21, 2012

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Nov 21, 2012

From @FROGGS

patch​:
rakudo/rakudo#87

@p6rt
Copy link
Author

p6rt commented Nov 21, 2012

From @FROGGS

the patch is maybe invalid.

std parses a do...while now, so maybe rakudo has to support it too.

@p6rt
Copy link
Author

p6rt commented Feb 1, 2015

From @skids

Now STD does not parse "do...while" anymore, and there
are tests to ensure rakudo does not parse "do...while"
either, and tests to ensure the "repeat...while" and
"repeat...until" forms work, as well.

S04-statements/repeat.t
S04-statements/do.t

So this might be a closeable ticket now?

@p6rt
Copy link
Author

p6rt commented Feb 1, 2015

From @masak

bri (>)​:

So this might be a closeable ticket now?

Yes, I would say so. Good work.

Resolving.

@p6rt p6rt closed this as completed Feb 1, 2015
@p6rt
Copy link
Author

p6rt commented Feb 1, 2015

@masak - Status changed from 'open' to 'resolved'

@p6rt p6rt added the Bug label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant