Page MenuHomePhabricator

[comm-lib] Use futures_util::Ready to simplify code
ClosedPublic

Authored by bartek on Jun 13 2024, 12:53 AM.
Tags
None
Referenced Files
F2831583: D12411.diff
Sat, Sep 28, 12:28 AM
Unknown Object (File)
Mon, Sep 9, 9:32 PM
Unknown Object (File)
Mon, Sep 9, 6:24 PM
Unknown Object (File)
Mon, Sep 9, 5:49 PM
Unknown Object (File)
Mon, Sep 9, 12:43 PM
Unknown Object (File)
Mon, Sep 9, 4:09 AM
Unknown Object (File)
Wed, Sep 4, 5:22 AM
Unknown Object (File)
Wed, Sep 4, 3:01 AM
Subscribers

Details

Summary

Minor change that greatly simplifies future code in the stacks.

The std::future::Ready::into_inner() is unstable so we had to use async { } block and boxed future.
Used futures_util::Ready instead, which has this method stable.

Test Plan

Cargo build

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage