Remove the `Futures()` method
9319f6d50d52
Actions

Authored by Joshua Spence <josh@joshuaspence.com> on Dec 30 2014, 13:38.

Description

Remove the Futures() method

Summary: Fixes T6829. Removes the Futures() method in favor of calling new FutureIterator($futures) explicitly. Depends on D11077, D11078 and D11079.

Test Plan: We could possibly deprecate this function instead (with phutil_deprecated).

Reviewers: epriestley, #blessed_reviewers

Reviewed By: epriestley, #blessed_reviewers

Subscribers: Korvin, epriestley

Maniphest Tasks: T6829

Differential Revision: https://secure.phabricator.com/D11081

Details

Committed

Joshua Spence <josh@joshuaspence.com>

Dec 30 2014, 13:39

Pushed

aubort

Mar 17 2017, 12:03

Parents

rPHUe6fef2c60a35: Use `new FutureIterator` instead of `Futures`

Branches

Unknown

Tags

Unknown

Event Timeline

Joshua Spence <josh@joshuaspence.com> committed rPHU9319f6d50d52: Remove the `Futures()` method (authored by Joshua Spence <josh@joshuaspence.com>).Dec 30 2014, 13:39