Skip to content
This repository has been archived by the owner on Oct 19, 2024. It is now read-only.

fix: replace FilterStream with concrete type #69

Merged
merged 3 commits into from
Sep 23, 2020
Merged

Conversation

gakonst
Copy link
Owner

@gakonst gakonst commented Sep 22, 2020

By passing the provider by reference and explicitly calling get_filter_changes inside the future impl we're able to simplify the function signature for anything involving FilterWatcher.

This allows us to remove the FilterStream helper trait (which was used to have simpler function signatures).

@gakonst gakonst merged commit bf1d1e0 into master Sep 23, 2020
@gakonst gakonst deleted the simplify-streams branch September 23, 2020 08:04
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.

1 participant