From 0ba134d09659a274d3c1dbcb3ca62a8f79039dde Mon Sep 17 00:00:00 2001 From: Arsham Skrenes Date: Wed, 10 Apr 2019 16:11:02 -0600 Subject: [PATCH 1/2] Properly allocate string and remove warning The former way generates the following warning: ISO C++ forbids converting a string constant to 'char*' This change makes a character array the size of the string with null ending. It's clearer and gets rid of the warning. --- libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino b/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino index 4ed8d52f5b6..ed2ed524a21 100644 --- a/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino +++ b/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino @@ -51,7 +51,7 @@ void InitESPNow() { // config AP SSID void configDeviceAP() { - char* SSID = "Slave_1"; + char SSID[] = "Slave_1"; bool result = WiFi.softAP(SSID, "Slave_1_Password", CHANNEL, 0); if (!result) { Serial.println("AP Config failed."); From 3b3533ee5bade72c0768a9365bde780361edcbd4 Mon Sep 17 00:00:00 2001 From: Arsham Skrenes Date: Thu, 11 Apr 2019 10:14:52 -0600 Subject: [PATCH 2/2] Better way Since this is technically immutable, the type should reflect this too. --- libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino b/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino index ed2ed524a21..d9029e961e3 100644 --- a/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino +++ b/libraries/ESP32/examples/ESPNow/Basic/Slave/Slave.ino @@ -51,7 +51,7 @@ void InitESPNow() { // config AP SSID void configDeviceAP() { - char SSID[] = "Slave_1"; + const char *SSID = "Slave_1"; bool result = WiFi.softAP(SSID, "Slave_1_Password", CHANNEL, 0); if (!result) { Serial.println("AP Config failed.");