Skip to content
This repository has been archived by the owner on Aug 19, 2021. It is now read-only.

Rename events.h -> mbed_events.h to avoid potential name collisions #7

Merged
merged 1 commit into from
Jul 30, 2016

Conversation

geky
Copy link
Contributor

@geky geky commented Jul 28, 2016

Note from @pan-:

Headers in this library should be prefixed otherwise they can easily clash with over headers.
For instance events.h is too generic and can be taken by other libraries.

The events.h name may be a common name for header files. On faulty build systems, there is a potential for this common name to cause the wrong header file to be included.

discussion #6
cc @pan-, @0xc0170
fixes #6

@geky geky mentioned this pull request Jul 28, 2016
@0xc0170
Copy link

0xc0170 commented Jul 29, 2016

+1

The events.h name may be a common name for header files. On faulty
build systems, there is a potential for this common name to cause
the wrong header file to be included.

discussion #6
@geky geky merged commit e605f0d into master Jul 30, 2016
@geky geky deleted the rename-mbed-events-h branch July 30, 2016 00:08
geky added a commit that referenced this pull request Aug 5, 2016
Rename the events library to the equeue library
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Avoid name clashing
2 participants