Compatibility with arduino-esp32 v3.0.0 : BLECharacteristic getValue std::string vs. String

[PaulZC]

Hi Avinab (@avinabmalla ),

Espressif made a breaking change to BLECharacteristic in this pull:

espressif/arduino-esp32#8724

This means this line fails to compile with v3.0.0 of the core:

ESP32_BleSerial/src/BleSerial.cpp

Line 167 in 9250e67

std::string value = pCharacteristic->getValue();

Will you accept a Pull Request to change std::string to String? Or is there a better way to solve this?

Thanks!
Paul (SparkFun)

[PaulZC]

Both std::string and String have c_str() and length(). So possible solutions are:

		auto value = pCharacteristic->getValue();

		String value = String(pCharacteristic->getValue().c_str(), pCharacteristic->getValue().length());

[avinabmalla]

Fixed in v1.0.5