Turn on a bunch of warnings
Turn on as many warnings as easily possible from:
https://github.com/lefticus/cppbestpractices/blob/e73393f25a85f83fed7399d8b65cb117d00b2231/02-Use_the_Tools_Available.md#L100
Tested:
ipmitool sensor list still works
Change-Id: Ied8fa66de9fcd25e448f8048c4f8216b426b6f55
Signed-off-by: James Feist <james.feist@linux.intel.com>
diff --git a/src/MCUTempSensor.cpp b/src/MCUTempSensor.cpp
index fd45327..0b621d0 100644
--- a/src/MCUTempSensor.cpp
+++ b/src/MCUTempSensor.cpp
@@ -105,7 +105,6 @@
{
std::string i2cBus = "/dev/i2c-" + std::to_string(busId);
int fd = open(i2cBus.c_str(), O_RDWR);
- size_t i = 0;
if (fd < 0)
{
@@ -172,7 +171,7 @@
double v = static_cast<double>(temp) / 1000;
if constexpr (debug)
{
- std::cerr << "Value update to " << (double)v << "raw reading "
+ std::cerr << "Value update to " << v << "raw reading "
<< static_cast<int>(temp) << "\n";
}
updateValue(v);
@@ -264,7 +263,7 @@
"GetManagedObjects");
}
-int main(int argc, char** argv)
+int main()
{
boost::asio::io_service io;
auto systemBus = std::make_shared<sdbusplus::asio::connection>(io);
@@ -276,7 +275,7 @@
boost::asio::deadline_timer configTimer(io);
std::function<void(sdbusplus::message::message&)> eventHandler =
- [&](sdbusplus::message::message& message) {
+ [&](sdbusplus::message::message&) {
configTimer.expires_from_now(boost::posix_time::seconds(1));
// create a timer because normally multiple properties change
configTimer.async_wait([&](const boost::system::error_code& ec) {