-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
Corrections to errors in the existing files #795
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
Conversation
@cclauss seems like the cpplint check tool is incorrectly linking the files: link
|
Too many files to review on one pull request. Can someone else please review? |
There are a lot of existing errors in the repository that need to be fixed. |
included in #805 |
Description of Change
There are indeed a lot of corrections because of the errors that already exist in the repository.
This is a major effort to fix all the existing errors in the repository:
There are a lot of files that changed and I already have a few auto-check tools that verify the CPPLINT and also compilation on linux, Windows and Mac operating systems. This can be checked here: https://github.com/kvedala/C-Plus-Plus/tree/cmake
Checklist
Notes: