[PATCH] Checked sourcecode with cppcheck
|Reported by:||Mr. X||Owned by:||ben|
|Priority:||Nice to Have||Milestone:||Alpha 15|
I checked the sourcecode of 0ad with cppcheck (current git head) once again (#606). I'll attach the log.
The majority of the message are complaints about uninitialized variables in constructor, variable scopes that can be reduced, about C-style pointer castings and functions that can be made const or static. However, among the "conclusive Messages", there are also 29 performance messages, three errors and lots of warnings. I did not verify that all these messages are correct, so there might be a few false positives.
I attached three log files:
- 0ad_full.txt contains all messages from cppcheck.
- 0ad_conclusive.txt contains all messages flagged as "conclusive"
- 0ad_cleaned.txt contains all conclusive message without the three most often shown messages, to make it easier for you to spot the other ones
Change History (27)
comment:4 by , 10 years ago
|Keywords:||review patch added|
|Summary:||Checked sourcecode with cppcheck → [PATCH] Checked sourcecode with cppcheck|