diff options
author | Ed Tanous <ed@tanous.net> | 2024-03-20 22:27:06 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-03-25 21:29:19 +0300 |
commit | f79ce6a8f745f0baa5ba457a5a0c85a9f549893f (patch) | |
tree | f0093a63a3fabef73d1bb231d764cc8a53b521c6 | |
parent | 47f2934cc509e420e5617a96158e76829746b835 (diff) | |
download | bmcweb-f79ce6a8f745f0baa5ba457a5a0c85a9f549893f.tar.xz |
Enable bugprone clang check
bugprone-multi-level-implicit-pointer-conversion is something that we
pass currently, with one exception in the deprecated rest API. Ignore
the one exception, as it's not clear how to fix it, and enable the
check.
Tested: Clang tidy passes.
Change-Id: Idc10e0bb7b876e1c70afa28f9c27cc7bef1db0d7
Signed-off-by: Ed Tanous <ed@tanous.net>
-rw-r--r-- | .clang-tidy | 1 | ||||
-rw-r--r-- | include/openbmc_dbus_rest.hpp | 5 |
2 files changed, 5 insertions, 1 deletions
diff --git a/.clang-tidy b/.clang-tidy index 71e80a4eed..c7963c089e 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -26,6 +26,7 @@ bugprone-misplaced-operator-in-strlen-in-alloc, bugprone-misplaced-pointer-arithmetic-in-alloc, bugprone-misplaced-widening-cast, bugprone-move-forwarding-reference, +bugprone-multi-level-implicit-pointer-conversion, bugprone-multiple-statement-macro, bugprone-narrowing-conversions, bugprone-no-escape, diff --git a/include/openbmc_dbus_rest.hpp b/include/openbmc_dbus_rest.hpp index e801a3291c..dd1dac37d9 100644 --- a/include/openbmc_dbus_rest.hpp +++ b/include/openbmc_dbus_rest.hpp @@ -951,7 +951,10 @@ int readMessageItem(const std::string& typeCode, sdbusplus::message_t& m, nlohmann::json& data) { T value; - + // When T == char*, this warning fires. Unclear how to resolve + // Given that sd-bus takes a void pointer to a char*, and that's + // Not something we can fix. + // NOLINTNEXTLINE(bugprone-multi-level-implicit-pointer-conversion) int r = sd_bus_message_read_basic(m.get(), typeCode.front(), &value); if (r < 0) { |