tree 1ea292df4aeb7850c4ca16789a6cf0f7e845ee17
parent 7c66cbd279304686afc34372c223a5dd8ad94395
author Craig Tiller <craig.tiller@gmail.com> 1595892089 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1595892089 +0000

[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: I7de6b80034a3426d69bbf38a9edbca62ca0319de
Reviewed-on: https://fuchsia-review.googlesource.com/c/experiences/+/410914
Testability-Review: Craig Tiller <ctiller@google.com>
Testability-Review: Chase Latta <chaselatta@google.com>
Reviewed-by: Chase Latta <chaselatta@google.com>
Reviewed-by: Kevin Lindkvist <lindkvist@google.com>
Commit-Queue: Craig Tiller <ctiller@google.com>