Remove null driver check + \n after drv->printSensorInfo()

tyeth · tyeth · commit 74eab4cd1cf7 · 2025-05-13T16:20:55.000+01:00
diff --git a/src/components/i2c/controller.cpp b/src/components/i2c/controller.cpp
@@ -1033,8 +1033,6 @@ void I2cController::ConfigureMuxChannel(uint32_t mux_channel, bool is_alt_bus) {
 void I2cController::PrintAllDrivers() {
   WS_DEBUG_PRINTLN("[i2c] Printing all drivers...");
   for (drvBase *drv : _i2c_drivers) {
-    if (drv == nullptr)
-      continue; // skip null drivers
     drv->printSensorInfo();
   }
 }
diff --git a/src/components/i2c/drivers/drvBase.h b/src/components/i2c/drivers/drvBase.h
@@ -868,7 +868,7 @@ class drvBase {
     WS_DEBUG_PRINT("\tSensor Period: ");
     WS_DEBUG_PRINT(_sensor_period);
     WS_DEBUG_PRINT("\tSensor Period remaining: ");
-    WS_DEBUG_PRINT(millis() - _sensor_period_prv);
+    WS_DEBUG_PRINTLN(millis() - _sensor_period_prv);
   }
 
 protected:

Original file line number	Diff line number	Diff line change
`@@ -1033,8 +1033,6 @@ void I2cController::ConfigureMuxChannel(uint32_t mux_channel, bool is_alt_bus) {`
`1033`	`1033`	`void I2cController::PrintAllDrivers() {`
`1034`	`1034`	`WS_DEBUG_PRINTLN("[i2c] Printing all drivers...");`
`1035`	`1035`	`for (drvBase *drv : _i2c_drivers) {`
`1036`		`- if (drv == nullptr)`
`1037`		`- continue; // skip null drivers`
`1038`	`1036`	`drv->printSensorInfo();`
`1039`	`1037`	`}`
`1040`	`1038`	`}`
Original file line number	Diff line number	Diff line change
`@@ -868,7 +868,7 @@ class drvBase {`
`868`	`868`	`WS_DEBUG_PRINT("\tSensor Period: ");`
`869`	`869`	`WS_DEBUG_PRINT(_sensor_period);`
`870`	`870`	`WS_DEBUG_PRINT("\tSensor Period remaining: ");`
`871`		`- WS_DEBUG_PRINT(millis() - _sensor_period_prv);`
	`871`	`+ WS_DEBUG_PRINTLN(millis() - _sensor_period_prv);`
`872`	`872`	`}`
`873`	`873`
`874`	`874`	`protected:`