From 3c2ec8cccd115b85d5fc76b025bf81ede30834d4 Mon Sep 17 00:00:00 2001 From: Dan Goodliffe Date: Sun, 24 Jul 2022 15:12:48 +0100 Subject: Enable all cppcheck checks and fixup --- Jamroot.jam | 7 ++++--- libadhocutil/globalStatic.impl.h | 2 +- libadhocutil/plugins.impl.h | 2 ++ libadhocutil/unittests/Jamfile.jam | 1 + 4 files changed, 8 insertions(+), 4 deletions(-) diff --git a/Jamroot.jam b/Jamroot.jam index 87fee9b..9da91b4 100644 --- a/Jamroot.jam +++ b/Jamroot.jam @@ -33,9 +33,10 @@ project gcc,debug:-Wlogical-op gcc,debug:-Wuseless-cast coverage:on - tidy:bin/sys.h - tidy:bin/net.h - tidy:unittests/bin/lorem-ipsum.h + tidy:all + tidy:libadhocutil/bin/sys.h + tidy:libadhocutil/bin/net.h + tidy:libadhocutil/unittests/bin/lorem-ipsum.h tidy:boost-* tidy:bugprone-* tidy:bugprone-macro-parentheses diff --git a/libadhocutil/globalStatic.impl.h b/libadhocutil/globalStatic.impl.h index 61314ea..4845b33 100644 --- a/libadhocutil/globalStatic.impl.h +++ b/libadhocutil/globalStatic.impl.h @@ -29,7 +29,7 @@ namespace AdHoc { typename GlobalStatic::Ptr & GlobalStatic::instance() { - static Ptr _instance; + static Ptr _instance {}; return _instance; } } diff --git a/libadhocutil/plugins.impl.h b/libadhocutil/plugins.impl.h index 772d37a..90e3070 100644 --- a/libadhocutil/plugins.impl.h +++ b/libadhocutil/plugins.impl.h @@ -20,6 +20,8 @@ namespace AdHoc { const std::type_info & PluginOf::type() const { + static_assert(std::is_reference_v); + // cppcheck-suppress returnTempReference; (typeid returns a reference) return typeid(T); } diff --git a/libadhocutil/unittests/Jamfile.jam b/libadhocutil/unittests/Jamfile.jam index cb6e9e3..bbcf2e3 100644 --- a/libadhocutil/unittests/Jamfile.jam +++ b/libadhocutil/unittests/Jamfile.jam @@ -23,6 +23,7 @@ project tidy:bugprone-use-after-move tidy:hicpp-invalid-access-moved tidy:accessMoved + tidy:literalWithCharPtrCompare ; h lorem-ipsum : lorem-ipsum.txt ; -- cgit v1.2.3