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

Stylefix #1

Open
brabo opened this issue Jun 26, 2016 · 2 comments
Open

Stylefix #1

brabo opened this issue Jun 26, 2016 · 2 comments

Comments

@brabo
Copy link
Member

brabo commented Jun 26, 2016

By now we highly likely merged some bad style.
Style review and fix of the entire repo is called for, rather sooner than later, in order to have consistent style in the whole repo.

danielinux added a commit that referenced this issue Jun 26, 2016
F7rcc support from Sync-
@brabo
Copy link
Member Author

brabo commented Jun 27, 2016

Subpart: verify and unify ALL include guards. some are named LIBNAME_XXXX and some without LIBNAME. Consistency is key.

@brabo
Copy link
Member Author

brabo commented Jun 27, 2016

Subpart: fixed structure of files. Some header have first includes and then include guards, etc.

@brabo brabo self-assigned this Jul 4, 2016
kuldeepdhaka pushed a commit to kuldeepdhaka/unicore-mx that referenced this issue Oct 13, 2016
Merging in my i2c-touchscreen example
kuldeepdhaka pushed a commit to kuldeepdhaka/unicore-mx that referenced this issue Oct 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant