Page MenuHomePhabricator

Resolve all futures inside FutureIterator
ClosedPublic

Authored by epriestley on Feb 26 2020, 7:34 PM.

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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Feb 26 2020, 7:34 PM
epriestley requested review of this revision.Feb 26 2020, 7:35 PM
epriestley updated this revision to Diff 50103.Feb 26 2020, 7:41 PM
  • Also remove uncalled "checkException()" method.
This revision was not accepted when it landed; it landed in state Needs Review.Sun, Mar 29, 5:18 PM
This revision was automatically updated to reflect the committed changes.