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

feat(acked): add a non consuming method to the notice #934

Open
wants to merge 1 commit into
base: acked
Choose a base branch
from

Conversation

joshuachp
Copy link

Type of change

This will make it possible to not block while trying to wait for the acknowledgement of multiple packets. For example, you are storing the NoticeFutures in a Vec, and you want to find the first acknowledge packet.

This would be similar to the try_recv(&mut) of the inner channel.

Linked discussion comment #805 (comment)

Checklist:

  • Formatted with cargo fmt
  • Make an entry to CHANGELOG.md if it's relevant to the users of the library. If it's not relevant mention why.

This will make it possible to not block while trying to wait for the
acknowledgement of multiple packets. For example, you are storing the
`NoticeFutures` in a `Vec`, and you want to find the first acknowledge packet.

This would be similar to the `try_recv(&mut)` of the inner channel.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant