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

Looping over a HyperSeq in sink context does nothing (for <a b c>.hyper { say 2 }) #6435

Closed
p6rt opened this issue Aug 9, 2017 · 5 comments
Labels
regression Issue did not exist previously

Comments

@p6rt
Copy link

p6rt commented Aug 9, 2017

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

Searchable as RT131865$

@p6rt
Copy link
Author

p6rt commented Aug 9, 2017

From @AlexDaniel

Code​:
for <a b c>.hyper { say 2 }

Result (2015.09-2015.06)​:
2
2
2

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
(no output)

There seems to be nothing wrong with using a for loop on .hyper or .race, so it should work. It starts working if you somehow use the value, for example​:

Code​:
do for <a b c>.hyper { say 2 }

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
2
2
2

Bisectable points to rakudo/rakudo@9b0b9ef

@p6rt
Copy link
Author

p6rt commented Aug 10, 2017

From @AlexDaniel

IRC discussion​: https://irclog.perlgeek.de/perl6/2017-08-09#i_14989886

Turns out that .hyper/.race never worked reliably anyway, so this ticket is less important than it looks.

Maybe we should have a meta ticket that tracks all .hyper/.race issues. Another option would be to just close them all, but this will leave us without a good list of tests that should be written.

On 2017-08-08 18​:40​:48, alex.jakimenko@​gmail.com wrote​:

Code​:
for <a b c>.hyper { say 2 }

Result (2015.09-2015.06)​:
2
2
2

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
(no output)

There seems to be nothing wrong with using a for loop on .hyper or
.race, so it should work. It starts working if you somehow use the
value, for example​:

Code​:
do for <a b c>.hyper { say 2 }

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
2
2
2

Bisectable points to
rakudo/rakudo@9b0b9ef

@p6rt
Copy link
Author

p6rt commented Oct 18, 2017

From @jnthn

On Tue, 08 Aug 2017 18​:40​:48 -0700, alex.jakimenko@​gmail.com wrote​:

Code​:
for <a b c>.hyper { say 2 }

Result (2015.09-2015.06)​:
2
2
2

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
(no output)

There seems to be nothing wrong with using a for loop on .hyper or
.race, so it should work. It starts working if you somehow use the
value, for example​:

Code​:
do for <a b c>.hyper { say 2 }

Result (9b0b9effe5,2017.07,HEAD(6745517))​:
2
2
2

Bisectable points to
rakudo/rakudo@9b0b9ef

Fixed in new hyper/race implementation; tests in S07-hyperrace/hyper.t and S07-hyperrace/race.t.

@p6rt
Copy link
Author

p6rt commented Oct 18, 2017

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

@p6rt
Copy link
Author

p6rt commented Oct 18, 2017

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

@p6rt p6rt closed this as completed Oct 18, 2017
@p6rt p6rt added the regression Issue did not exist previously label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Issue did not exist previously
Projects
None yet
Development

No branches or pull requests

1 participant