OneButton icon indicating copy to clipboard operation
OneButton copied to clipboard

Add cppcheck

Open mkinney opened this issue 3 years ago • 4 comments

See https://github.com/danmar/cppcheck/ or https://cppcheck.sourceforge.io/ for more info about cppcheck

~~and for the github action:~~

~~https://github.com/marketplace/actions/cppcheck-action~~

I excluded the "knownConditionTrueFalse" in the suppressions.txt file. May want to look into that.

mkinney avatar Jan 27 '22 07:01 mkinney

I have not tested that github action, so it make take some tweaking.

mkinney avatar Jan 27 '22 07:01 mkinney

I was able to test the github action on my repo.

Let me know if you have any questions/suggestions/comments.

mkinney avatar Jan 27 '22 07:01 mkinney

Looks interesting. When you have finished the settings I will merge.

(Already saw one of your the results ([knownConditionTrueFalse]) This is usually removed by the optimizing compiler so is not in effect but surely can be cleaned up.

mathertel avatar Jan 27 '22 09:01 mathertel

I’m finished with it, so please merge if ok.

I would be grateful if the knownConditionTrueFalse could be fixed at some point.

thanks

mkinney avatar Jan 27 '22 10:01 mkinney