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

fix some warnings for g++ #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

alexrayne
Copy link

hallow!
here provided fixes about some warnings a got when build contiky for cc1310 with this lib

1)move align attributes on typedefs, so that g++ accepts it with no warning
2)read registers with no effect - escape warning of no effect via (void) cast
3)provide warning supression for case with no break via comments mentioned by gcc7 and eclipse

alexrayne added 2 commits August 9, 2017 11:46
…g++ accepts it with no warning

!        :reg read with no effect - escape warning of no effect via (void) cast
* gcc will no warns no effect on volatile vars access, but some code analysis do.
mark such registers reads explicilty for cade analysis tool happy.
@alexrayne alexrayne closed this Aug 9, 2017
@alexrayne
Copy link
Author

something wrong here

@alexrayne alexrayne reopened this Aug 10, 2017
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.

1 participant