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

L2CAP channel could get released while waiting #201

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

kext
Copy link
Contributor

@kext kext commented Nov 6, 2023

I run into the unreachable! statement when I unplug the device on the other side.

Maybe this should error out directly, but this was the easiest way to fix the problem.
On the next call to try_tx after continue the connection is checked and returns DisconnectedError.

Copy link
Member

@Dirbaio Dirbaio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@Dirbaio Dirbaio added this pull request to the merge queue Nov 8, 2023
Merged via the queue into embassy-rs:master with commit 17c6c2b Nov 8, 2023
2 checks passed
@kext kext deleted the l2cap-tx-channel-released branch November 8, 2023 23:13
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.

2 participants