search

zesterer / flume

A safe and fast multi-producer, multi-consumer channel.
https://crates.io/crates/flume
Apache License 2.0
2.47k stars 85 forks source link

[feat] add shrink_to_fit() to Sender<T> and Receiver<T> #147

Closed rakbladsvalsen closed 7 months ago

rakbladsvalsen commented 7 months ago

Fixes https://github.com/zesterer/flume/issues/146

rakbladsvalsen commented 7 months ago

Note: I ran cargo fmt on the entire repo, so it might look like I made a lot of changes but I actually only added a few lines of code. Clippy is also complaining but I didn't bother fixing the clippy warnings.

zesterer commented 7 months ago

Note: I ran cargo fmt on the entire repo

This is unfortunate and makes it quite difficult to review this PR. Did you have the relevant changes in isolation?

rakbladsvalsen commented 7 months ago

This is unfortunate and makes it quite difficult to review this PR. Did you have the relevant changes in isolation?

Yeah, I'll close this PR and create another one with only the relevant changes.