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

Simple gather loop flusters Rakudo #2732

Closed
p6rt opened this issue Apr 28, 2012 · 5 comments
Closed

Simple gather loop flusters Rakudo #2732

p6rt opened this issue Apr 28, 2012 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Apr 28, 2012

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

Searchable as RT112654$

@p6rt
Copy link
Author

p6rt commented Apr 28, 2012

From @colomon

colomon​: r​: sub mcf() { gather loop { take 1; last; }; }; say mcf().perl
p6eval​: rakudo dda5b0​: OUTPUT«Method 'eager' not found for invocant of
class 'Integer'␤ in sub coro at src/gen/CORE.setting​:4926␤ in method
reify at src/gen/CORE.setting​:4907␤ in method reify at
src/gen/CORE.setting​:4678␤ in method gimme at
src/gen/CORE.setting​:5065␤ in method perl at src/…

--
Solomon Foster​: colomon@​gmail.com
HarmonyWare, Inc​: http://www.harmonyware.com

@p6rt
Copy link
Author

p6rt commented Apr 28, 2012

From @pmichaud

On Fri, Apr 27, 2012 at 07​:59​:22PM -0700, Solomon Foster wrote​:

colomon​: r​: sub mcf() { gather loop { take 1; last; }; }; say mcf().perl
p6eval​: rakudo dda5b0​: OUTPUT«Method 'eager' not found for invocant of
class 'Integer'␤ in sub coro at src/gen/CORE.setting​:4926␤ in method
reify at src/gen/CORE.setting​:4907␤ in method reify at
src/gen/CORE.setting​:4678␤ in method gimme at
src/gen/CORE.setting​:5065␤ in method perl at src/…

The problem was actually with the 'loop' statement, not the 'gather';
the 'loop' statement was returning a Parrot Integer 1. Notably
the gather statement works fine if 'loop' is replaced with 'while 1'.

Now fixed in a25e7d1, needs spectests to close ticket.

Pm

@p6rt
Copy link
Author

p6rt commented Apr 28, 2012

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

@p6rt
Copy link
Author

p6rt commented Apr 28, 2012

From @moritz

Tested in S04-statements/loop.t

@p6rt
Copy link
Author

p6rt commented Apr 28, 2012

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant