Skip to content

Fix some Cppcheck findings, makes several variable more local

Christoph Grüninger requested to merge (removed):feature/fix-cppcheck-findings into master

The last commit assumes, C99 or newer is used. It looks like it was written to be C89 or C90 on purpose. Is this still the right approach? I propose to allow C11. The CMakeLists.txt does not mention C at all, not in project and no enable_language. Maybe this should be done, too?

If you are not convinced to move to C99/C11, the last commit can be dropped.

Topic-rename: fix-cppcheck-findings

Edited by Brad King

Merge request reports