Enable cppcoreguidelines-pro-type-vararg check
We only had one usage of printf in the code that was in violation of
this rule, so replace it with iostreams, and enable the check.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ie62165b599a996f34893aa5a3f8d1f6e6cbaf903
diff --git a/src/MCUTempSensor.cpp b/src/MCUTempSensor.cpp
index ede2a5e..4653bc8 100644
--- a/src/MCUTempSensor.cpp
+++ b/src/MCUTempSensor.cpp
@@ -102,8 +102,9 @@
int MCUTempSensor::getMCURegsInfoWord(uint8_t regs, int16_t* pu16data)
{
std::string i2cBus = "/dev/i2c-" + std::to_string(busId);
- int fd = open(i2cBus.c_str(), O_RDWR);
+ // NOLINTNEXTLINE(cppcoreguidelines-pro-type-vararg)
+ int fd = open(i2cBus.c_str(), O_RDWR);
if (fd < 0)
{
std::cerr << " unable to open i2c device" << i2cBus << " err=" << fd
@@ -111,6 +112,7 @@
return -1;
}
+ // NOLINTNEXTLINE(cppcoreguidelines-pro-type-vararg)
if (ioctl(fd, I2C_SLAVE_FORCE, mcuAddress) < 0)
{
std::cerr << " unable to set device address\n";
@@ -119,6 +121,7 @@
}
unsigned long funcs = 0;
+ // NOLINTNEXTLINE(cppcoreguidelines-pro-type-vararg)
if (ioctl(fd, I2C_FUNCS, &funcs) < 0)
{
std::cerr << " not support I2C_FUNCS\n";