commit 7c66cbd279304686afc34372c223a5dd8ad94395
author Yaneury Fermin <yaneury@google.com> Mon Jul 27 19:06:58 2020 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Mon Jul 27 19:06:58 2020 +0000
tree 0d7096e29513556835c949ac5857956177d80b5c
parent 77dacb00187497d10d17fcffa91aa6027cfa5b3a

[rust][fasync] Prepare for the removal of fasync::spawn*

TLDR: fasync::spawn functions are going away in preference of the new
Task::spawn API's. This CL prepares the way by replacing usage of the old
API's with usage of the new one.

We're deprecating and replacing the old spawn functions (spawn, spawn_local,
spawn_blocking) and replacing them with an API centered around Task objects.

Tasks own the execution of the underlying future, and if dropped will cancel
that execution. To regain the old, unowned execution that spawn API's offered
a detach method is offered.

This change series sweeps through the codebase and replaces spawn() with
Task::spawn().detach() and spawn_local() with Task::local().detach() in
preparation for the removal of the original API's.

It's hoped that code maintainers will then take the opportunity (when it makes
sense for them to do so) to move to a model where Tasks are not detached but
owned for as long as they need to be running.

Change-Id: I4324e0217458bbda1e7d15f047e4318ed921c55e
Reviewed-on: https://fuchsia-review.googlesource.com/c/experiences/+/410756
Reviewed-by: Chase Latta <chaselatta@google.com>
Testability-Review: Chase Latta <chaselatta@google.com>
Commit-Queue: Yaneury Fermin <yaneury@google.com>