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

Implemented the embedded_serial traits. #74

Closed
wants to merge 1 commit into from
Closed

Implemented the embedded_serial traits. #74

wants to merge 1 commit into from

Conversation

thejpster
Copy link

I've implemented the embedded serial traits we talked about during novemb.rs. Untested, as I don't have a F3 Discovery...

@japaric
Copy link
Owner

japaric commented Jan 7, 2017

Thanks for the PR, @thejpster. Unfortunately the serial::Port struct is not public and thus it's not exposed as part of the API so users won't be able to use these traits even if we implement them for serial::Port. :-/

However, an interesting experiment would be implementing these traits for the future based serial API in #52 to provide a more ergonomic blocking API.

@homunkulus
Copy link
Collaborator

☔ The latest upstream changes (presumably e2fecfb) made this pull request unmergeable. Please resolve the merge conflicts.

@japaric
Copy link
Owner

japaric commented Jun 22, 2018

I believe this no longer applies now that we have embedded-hal.

@japaric japaric closed this Jun 22, 2018
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.

3 participants