ARMmbed / ARMmbed/mbed-drivers

mbed.h imports namespaces

Open
#171 2 comments 0 reactions 0 assignees View on GitHub
mirrored
Dominant language
C++
Stars
39
Forks
41
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Read mbed-drivers/mbed.h at the two linked locations, lines 63-64, to inspect the namespace imports. Done means mbed.h no longer imports namespaces and the change is treated as the stated breaking change requiring a new major version; also review the omni-include guidance in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
embedded-iot
Issue type
Refactor
Difficulty
1/5
Estimated time
Under an hour
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.