-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Expose the public Stream API #14212
Expose the public Stream API #14212
Conversation
Stream.h contains a public API which is not currently exposed via mbed.h (when it should be). This commit fixes this.
@adbridge, thank you for your changes. |
This class is public API, it used to be exposed via Serial.h. LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. Since this API is public, it should be exposed in mbed.h
This pull request has automatically been marked as stale because it has had no recent activity. @ARMmbed/mbed-os-maintainers, please start CI to get the PR merged. |
CI started |
Jenkins CI Test : ✔️ SUCCESSBuild Number: 1 | 🔒 Jenkins CI Job | 🌐 Logs & ArtifactsCLICK for Detailed Summary
|
Summary of changes
Stream.h contains a public API which is not currently exposed via
mbed.h (when it should be). This commit fixes this.
Fixes #13354
Impact of changes
The stream API will now be available by including mbed.h.
Migration actions required
None
Documentation
Pull request type
Test results
Reviewers