Ping ESP is now compatible with newer Arduino SDK

Author: sblantipodi

library.json

@@ -6,7 +6,7 @@
         "type": "git",
         "url": "https://github.com/sblantipodi/arduino_bootstrapper.git"
     },
-    "version": "1.12.5",
+    "version": "1.12.6",
     "examples": "examples/*.cpp", 
     "exclude": "tests",
     "frameworks": "arduino",

library.properties

@@ -1,5 +1,5 @@
 name=Bootstrapper
-version=1.12.5
+version=1.12.6
 author=Davide Perini <[email protected]>
 maintainer=Davide Perini <[email protected]>
 sentence=A client library for MQTT messaging.

src/PingESP.cpp

@@ -27,11 +27,15 @@ extern "C" void esp_schedule();
 extern "C" void esp_yield();
 extern "C" void __esp_suspend();
 
+/**
+ * Newer Arduino SDK than 3.0.2 does esp_schedule() inside esp_yield(), this break ping functions here.
+ * Direct alternative would be to use new esp_suspend().
+ * Continue to use esp_yield() implementation when there is no esp_suspend() function (<= 3.0.2),
+ * use new esp_suspend() instead (> 3.0.2).
+ */
 extern "C" void suspend_or_yield() {
-  Serial.printf("This is the default handler\n");
   esp_yield();
 }
-
 void __esp_suspend(void) __attribute__((weak, alias("suspend_or_yield")));
 
 PingESP::PingESP() {}
@@ -80,10 +84,4 @@ void PingESP::receivePingCallback(void *opt, void *resp) {
     esp_schedule();
   }
 }
-
-///**
-// * Weak function, if there is some implementation of esp_suspend() elsewhere, use that implementation,
-// * use this implementation instead.
-// */
-
 #endif