A bug in Future impl of Recv

smol-rs / async-broadcast

Async broadcast channels

Apache License 2.0

160 stars 26 forks source link

A bug in Future impl of Recv #17

Closed zeenix closed 3 years ago

zeenix commented 3 years ago

I tried to (manually) implement a Stream where the poll_next impl. would in turn poll the Future created from select of async_broadcast::Recv and async_channel::Recv but it'd lock up and not getting new messages. I changed the impl. to use <Receiver as StreamNext>::next and it worked just fine.

I will try to create a simple testcase for this but just wanted to create an issue first so I don't forget.

zeenix commented 3 years ago

Turned out to be a case of accidental drop of Future in my Stream impl. :facepalm: