Replaced error message for double characteristics

thomasvdwege · web-flow · commit 247d66da9282 · 2018-03-17T14:45:38.000+01:00
Made the fix that @chegewara proposed. I was confused by the error and found this github issue in the repository: nkolban#175
diff --git a/cpp_utils/BLEService.cpp b/cpp_utils/BLEService.cpp
@@ -192,7 +192,7 @@ void BLEService::addCharacteristic(BLECharacteristic* pCharacteristic) {
 
 	// Check that we don't add the same characteristic twice.
 	if (m_characteristicMap.getByUUID(pCharacteristic->getUUID()) != nullptr) {
-		ESP_LOGE(LOG_TAG, "<< Attempt to add a characteristic but we already have one with this UUID");
+		ESP_LOGW(LOG_TAG, "<< Adding a new characteristic with the same UUID as a previous one");
 		//return;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -192,7 +192,7 @@ void BLEService::addCharacteristic(BLECharacteristic* pCharacteristic) {`
`192`	`192`
`193`	`193`	`// Check that we don't add the same characteristic twice.`
`194`	`194`	`if (m_characteristicMap.getByUUID(pCharacteristic->getUUID()) != nullptr) {`
`195`		`- ESP_LOGE(LOG_TAG, "<< Attempt to add a characteristic but we already have one with this UUID");`
	`195`	`+ ESP_LOGW(LOG_TAG, "<< Adding a new characteristic with the same UUID as a previous one");`
`196`	`196`	`//return;`
`197`	`197`	`}`
`198`	`198`