Fix clang-tidy header check
Apparently our exclusion of the linux headers covered up some issues in
our own headers, and was erroneously ignoring all headers, not just the
linux headers.
Apparently this now detects recursion quite a bit, so disable that check
for the moment, as well as the special member variables checks. Also,
disable some cert checks that are duplicates of
bugprone-reserved-identifier, because of the aformentioned ignored linux
headers
All other changes are done automatically.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ie640495146dbe155a85d8a9e81be78539137a88b
diff --git a/src/NVMeSensorMain.cpp b/src/NVMeSensorMain.cpp
index 8eaeb57..c2b32f0 100644
--- a/src/NVMeSensorMain.cpp
+++ b/src/NVMeSensorMain.cpp
@@ -132,7 +132,7 @@
for (const auto& [interfacePath, sensorData] : sensorConfigurations)
{
// find base configuration
- auto sensorBase = sensorData.find(NVMeSensor::CONFIG_TYPE);
+ auto sensorBase = sensorData.find(NVMeSensor::configType);
if (sensorBase != sensorData.end())
{
const SensorBaseConfigMap& sensorConfig = sensorBase->second;
@@ -181,7 +181,7 @@
handleSensorConfigurations(io, objectServer, dbusConnection,
sensorConfigurations);
}));
- getter->getConfiguration(std::vector<std::string>{NVMeSensor::CONFIG_TYPE});
+ getter->getConfiguration(std::vector<std::string>{NVMeSensor::configType});
}
static void interfaceRemoved(sdbusplus::message::message& message,
@@ -257,7 +257,7 @@
static_cast<sdbusplus::bus::bus&>(*systemBus),
"type='signal',member='PropertiesChanged',path_namespace='" +
std::string(inventoryPath) + "',arg0namespace='" +
- std::string(NVMeSensor::CONFIG_TYPE) + "'",
+ std::string(NVMeSensor::configType) + "'",
eventHandler);
// Watch for entity-manager to remove configuration interfaces