Skip to content
This repository was archived by the owner on Oct 30, 2019. It is now read-only.

Update for futures v0.3.0-alpha.15 #18

Merged
merged 1 commit into from
Apr 29, 2019

Conversation

Nemo157
Copy link
Member

@Nemo157 Nemo157 commented Apr 27, 2019

Replace (most) usage of FutureObj with BoxFuture and use BoxFuture to
simplify some function signatures.

Enable the async-await feature for the tests.

Description

Motivation and Context

Fixes #6.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

The only breaking change is changing Runtime::spawn_obj into Runtime::spawn_boxed, the other function signature changes are identical after type alias expansion.

Replace (most) usage of FutureObj with BoxFuture and use BoxFuture to
simplify some function signatures.

Enable the async-await feature for the tests.
Copy link
Collaborator

@yoshuawuyts yoshuawuyts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is excellent, thanks so much!

@yoshuawuyts yoshuawuyts merged commit 5f25b1c into rustasync:master Apr 29, 2019
@Nemo157 Nemo157 deleted the futures-alpha.15 branch April 29, 2019 11:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move from StreamObj to Pin<Box<dyn Future>>
2 participants