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

mbed.h imports namespaces #171

Open
bremoran opened this issue Mar 3, 2016 · 2 comments
Open

mbed.h imports namespaces #171

bremoran opened this issue Mar 3, 2016 · 2 comments
Labels

Comments

@bremoran
Copy link
Collaborator

bremoran commented Mar 3, 2016

mbed.h should not import namespaces. This makes a decision about how to write programs for the developer instead of allowing them to make the decision. Changing this behaviour is a breaking change and will require a new major version.

https://github.com/ARMmbed/mbed-drivers/blob/master/mbed-drivers/mbed.h#L63
https://github.com/ARMmbed/mbed-drivers/blob/master/mbed-drivers/mbed.h#L64

In the mean time, users should be encouraged to include the API of only the portions of mbed drivers they are using, rather than all of mbed-drivers via the omni-include.

@bremoran
Copy link
Collaborator Author

bremoran commented Mar 3, 2016

cc @pan- @hugovincent @bogdanm @mjs-arm @0xc0170

@ciarmcom
Copy link
Member

ciarmcom commented Mar 3, 2016

ARM Internal Ref: IOTSFW-2205

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants