Page MenuHomePhabricator

Resolve all futures inside FutureIterator
ClosedPublic

Authored by epriestley on Feb 26 2020, 7:34 PM.
Tags
None
Referenced Files
F18789820: D21033.id.diff
Wed, Oct 15, 11:14 AM
F18787401: D21033.diff
Tue, Oct 14, 11:35 PM
F18782511: D21033.id50139.diff
Mon, Oct 13, 1:48 AM
F18782412: D21033.id50103.diff
Mon, Oct 13, 1:29 AM
F18782411: D21033.id50102.diff
Mon, Oct 13, 1:29 AM
F18781721: D21033.diff
Sun, Oct 12, 11:23 PM
F18780549: D21033.id.diff
Sun, Oct 12, 5:11 PM
F18773200: D21033.diff
Thu, Oct 9, 8:06 AM
Subscribers
None

Details

Summary

Depends on D21032. Ref T11968. Currently, "Future" and "FutureIterator" can both resolve futures.

Treat "Future->resolve()" as sugar on resolving an iterator of size 1.

Test Plan

Ran tests, created this revision.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

  • Also remove uncalled "checkException()" method.
This revision was not accepted when it landed; it landed in state Needs Review.Mar 29 2020, 5:18 PM
This revision was automatically updated to reflect the committed changes.