-
Notifications
You must be signed in to change notification settings - Fork 33
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
Clean up clang-tidy warnings #209
Comments
Right. We should solve them as soon as possible. |
I think the best way to address this is to create pull requests sorted by the type of clang-tidy warning. |
I will focus on this after imgui PR is merged. |
The issues which were easy to fix are solved in #250. |
This leaves us with the following issues: Warning G1510A767: Constructors/Destructors: Issues with rule of 0, 3 or 5
Warning G403A5994: Issues with class access specifiers and member visibility
Warning G403A5994: Issues with class access specifiers and member visibility
Warning G6B27326A: Implicitly deleted move assignment operators
Warning G75DF59F6
Warning GE4E5D495
Warning GAB3A7E52
Misc notes
|
All issues are fixed with the new pull request. |
There are a lot of warnings like unused variables, uninitialized variables, etc....
The text was updated successfully, but these errors were encountered: