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

Promise.allof() with empty argument is never kept #4969

Closed
p6rt opened this issue Dec 31, 2015 · 4 comments
Closed

Promise.allof() with empty argument is never kept #4969

p6rt opened this issue Dec 31, 2015 · 4 comments

Comments

@p6rt
Copy link

p6rt commented Dec 31, 2015

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

Searchable as RT127101$

@p6rt
Copy link
Author

p6rt commented Dec 31, 2015

From @smls

This hangs indefinitely​:

  my @​promises;
  await Promise.allof(@​promises);

Expected behavior​: The await() should return immediately.

That would not just be the logical thing to do, but also make it consistent with synchronous junctions​:

  my @​values;
  say all(@​values).Bool; # True

@p6rt
Copy link
Author

p6rt commented Dec 31, 2015

From @lizmat

On 31 Dec 2015, at 07​:29, Sam S. (via RT) <perl6-bugs-followup@​perl.org> wrote​:

# New Ticket Created by Sam S.
# Please include the string​: [perl #​127101]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl6/Ticket/Display.html?id=127101 >

This hangs indefinitely​:

my @​promises;
await Promise.allof(@​promises);

Expected behavior​: The await() should return immediately.

That would not just be the logical thing to do, but also make it consistent with synchronous junctions​:

my @​values;
say all(@​values).Bool; # True

Thanks for reporting!

Fixed with 37e998d , tests added with f153dc5 , can be closed.

Liz

@p6rt
Copy link
Author

p6rt commented Dec 31, 2015

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

@p6rt p6rt closed this as completed Dec 31, 2015
@p6rt
Copy link
Author

p6rt commented Dec 31, 2015

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

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

No branches or pull requests

1 participant