Skip to content

Commit b4751d1

Browse files
CppCheck report done
1 parent ba115c3 commit b4751d1

File tree

1 file changed

+20
-1
lines changed

1 file changed

+20
-1
lines changed

ProjectAnalysisReport.md

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,4 +95,23 @@ Library does not contain any memory leak all allocated memory was freed, the err
9595

9696
Place the script in root of `Simple graphs` and run it. Script will build the project if that is needed and run the tool. After the tool is finished it will output log file which will reperesent the analysis of the said tool.
9797

98-
---
98+
---
99+
100+
### [Cppcheck](https://cppcheck.sourceforge.io/)
101+
102+
#### Brief summary of the tool
103+
104+
`Cppcheck` is a static analysis tool for C/C++ code. It provides unique code analysis to detect bugs and focuses on detecting undefined behaviour and dangerous coding constructs. The goal is to have very few false positives.
105+
106+
#### Key results of analysis
107+
108+
There are a couple of `information` severity regarding missing includes, but CppCheck states that it is not a problem and that CppCheck does not need them to perform analysis.
109+
110+
`Style` severity is tied to all other messages, there is one note about constructor that should be explicit and all ohter messages are about unused functions.
111+
112+
#### Sugested changes
113+
114+
Since there was no message with high severity, no breaking error was found.
115+
All the messages are benign and unused function i would not want to remove from this library since there is possible future development.
116+
As for the constructor that was noted to be switched to explicit that is the ***that is the only pottentialy bad thing.***
117+
Since the noted constructor has only 1 argument it could potentialy lead to bad behaviour if its left without `explicit`.

0 commit comments

Comments
 (0)