fix peer removal

craiglink · web-flow · commit ab3082a88c62 · 2024-07-25T17:29:10.000-07:00
in the case were _esp_now_del_peer, the peer could never be added again because `added` would always be true and never reset.  This forces it to be reset
diff --git a/libraries/ESP_NOW/src/ESP32_NOW.cpp b/libraries/ESP_NOW/src/ESP32_NOW.cpp
@@ -306,12 +306,9 @@ bool ESP_NOW_Peer::remove() {
     return true;
   }
   log_v("Peer removed - " MACSTR, MAC2STR(mac));
+  added = false;
   esp_err_t err = _esp_now_del_peer(mac);
-  if (err == ESP_OK) {
-    added = false;
-    return true;
-  }
-  return false;
+  return (err == ESP_OK);
 }
 
 const uint8_t *ESP_NOW_Peer::addr() const {

Original file line number	Diff line number	Diff line change
`@@ -306,12 +306,9 @@ bool ESP_NOW_Peer::remove() {`
`306`	`306`	`return true;`
`307`	`307`	`}`
`308`	`308`	`log_v("Peer removed - " MACSTR, MAC2STR(mac));`
	`309`	`+ added = false;`
`309`	`310`	`esp_err_t err = _esp_now_del_peer(mac);`
`310`		`- if (err == ESP_OK) {`
`311`		`- added = false;`
`312`		`- return true;`
`313`		`- }`
`314`		`- return false;`
	`311`	`+ return (err == ESP_OK);`
`315`	`312`	`}`
`316`	`313`
`317`	`314`	`const uint8_t *ESP_NOW_Peer::addr() const {`