refactor(libraries/Wire): remove unused variables

safocl · safocl · commit 7a351dd0d0ba · 2024-01-18T17:01:03.000+04:00
Compiler reports "Wire.cpp:393:15: error: variable 'err' set but not used [-Werror=unused-but-set-variable]".
diff --git a/libraries/Wire/src/Wire.cpp b/libraries/Wire/src/Wire.cpp
@@ -390,7 +390,6 @@ uint32_t TwoWire::getClock()
 
 void TwoWire::setClock(uint32_t frequency)
 {
-    esp_err_t err = ESP_OK;
 #if !CONFIG_DISABLE_HAL_LOCKS
     //acquire lock
     if(lock == NULL || xSemaphoreTake(lock, portMAX_DELAY) != pdTRUE){
@@ -401,11 +400,10 @@ void TwoWire::setClock(uint32_t frequency)
 #if SOC_I2C_SUPPORT_SLAVE
     if(is_slave){
         log_e("Bus is in Slave Mode");
-        err = ESP_FAIL;
     } else 
 #endif /* SOC_I2C_SUPPORT_SLAVE */
     {
-        err = i2cSetClock(num, frequency);
+        i2cSetClock(num, frequency);
     }
 #if !CONFIG_DISABLE_HAL_LOCKS
     //release lock

Original file line number	Diff line number	Diff line change
`@@ -390,7 +390,6 @@ uint32_t TwoWire::getClock()`
`390`	`390`
`391`	`391`	`void TwoWire::setClock(uint32_t frequency)`
`392`	`392`	`{`
`393`		`- esp_err_t err = ESP_OK;`
`394`	`393`	`#if !CONFIG_DISABLE_HAL_LOCKS`
`395`	`394`	`//acquire lock`
`396`	`395`	`if(lock == NULL \|\| xSemaphoreTake(lock, portMAX_DELAY) != pdTRUE){`
`@@ -401,11 +400,10 @@ void TwoWire::setClock(uint32_t frequency)`
`401`	`400`	`#if SOC_I2C_SUPPORT_SLAVE`
`402`	`401`	`if(is_slave){`
`403`	`402`	`log_e("Bus is in Slave Mode");`
`404`		`- err = ESP_FAIL;`
`405`	`403`	`} else`
`406`	`404`	`#endif /* SOC_I2C_SUPPORT_SLAVE */`
`407`	`405`	`{`
`408`		`- err = i2cSetClock(num, frequency);`
	`406`	`+ i2cSetClock(num, frequency);`
`409`	`407`	`}`
`410`	`408`	`#if !CONFIG_DISABLE_HAL_LOCKS`
`411`	`409`	`//release lock`