Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add poll_recv_many for mpsc Receiver/UnboundedReceiver. #6209

Closed
leptonyu opened this issue Dec 11, 2023 · 1 comment · Fixed by #6236
Closed

Add poll_recv_many for mpsc Receiver/UnboundedReceiver. #6209

leptonyu opened this issue Dec 11, 2023 · 1 comment · Fixed by #6236
Labels
A-tokio Area: The main tokio crate C-feature-request Category: A feature request. E-help-wanted Call for participation: Help is requested to fix this issue. M-sync Module: tokio/sync

Comments

@leptonyu
Copy link

Is your feature request related to a problem? Please describe.
mpsc::Receiver recently add new method recv_many, but it doesn't expose poll version.

Describe the solution you'd like
Suggest to expose poll_recv_many.

Describe alternatives you've considered
(See above)

Additional context
(See above)

@leptonyu leptonyu added A-tokio Area: The main tokio crate C-feature-request Category: A feature request. labels Dec 11, 2023
@Darksonn Darksonn added M-sync Module: tokio/sync E-help-wanted Call for participation: Help is requested to fix this issue. labels Dec 11, 2023
@Darksonn
Copy link
Contributor

Darksonn commented Dec 12, 2023

I don't mind adding this. The implementation will be really simple, but it will need well-written documentation.

See the existing poll_recv method for inspiration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tokio Area: The main tokio crate C-feature-request Category: A feature request. E-help-wanted Call for participation: Help is requested to fix this issue. M-sync Module: tokio/sync
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants