switch to Definitions-based debug + error logging, normalize macro names

Author: abachman

src/AdafruitIO.cpp

@@ -29,15 +29,17 @@ AdafruitIO::AdafruitIO(const char *user, const char *key)
 
 void errorCallback(char *err, uint16_t len)
 {
-  AIO_ERR_PRINTLN();
-  AIO_ERR_PRINT("ERROR: ");
-  AIO_ERR_PRINTLN(err);
-  AIO_ERR_PRINTLN();
+  AIO_ERROR_PRINTLN();
+  AIO_ERROR_PRINT("ERROR: ");
+  AIO_ERROR_PRINTLN(err);
+  AIO_ERROR_PRINTLN();
 }
 
 void AdafruitIO::connect()
 {
 
+  AIO_DEBUG_PRINTLN("AdafruitIO::connect()");
+
   if(_err_sub) {
     // setup error sub
     _err_sub = new Adafruit_MQTT_Subscribe(_mqtt, _err_topic);
@@ -221,7 +223,11 @@ aio_status_t AdafruitIO::mqttStatus()
   // if the connection failed,
   // return so we don't hammer IO
   if(_status == AIO_CONNECT_FAILED)
+  {
+    AIO_ERROR_PRINT("mqttStatus() failed to connect");
+    AIO_ERROR_PRINTLN(_mqtt->connectErrorString(_status));
     return _status;
+  }
 
   if(_mqtt->connected())
     return AIO_CONNECTED;

src/AdafruitIO.h

@@ -30,36 +30,6 @@
   #error "This sketch requires Adafruit MQTT Library v0.17.0 or higher. Please install or upgrade using the Library Manager."
 #endif
 
-// Uncomment/comment to turn on/off debug output messages.
-// #define IO_DEBUG
-// Uncomment/comment to turn on/off error output messages.
-#define IO_ERROR
-
-// Set where debug messages will be printed.
-#define IO_DEBUG_PRINTER Serial
-// If using something like Zero or Due, change the above to SerialUSB
-
-// Define actual debug output functions when necessary.
-#ifdef IO_DEBUG
-  #define IO_DEBUG_PRINT(...) { DEBUG_PRINTER.print(__VA_ARGS__); }
-  #define IO_DEBUG_PRINTLN(...) { DEBUG_PRINTER.println(__VA_ARGS__); }
-  #define IO_DEBUG_PRINTBUFFER(buffer, len) { printBuffer(buffer, len); }
-#else
-  #define IO_DEBUG_PRINT(...) {}
-  #define IO_DEBUG_PRINTLN(...) {}
-  #define IO_DEBUG_PRINTBUFFER(buffer, len) {}
-#endif
-
-#ifdef IO_ERROR
-  #define IO_ERROR_PRINT(...) { DEBUG_PRINTER.print(__VA_ARGS__); }
-  #define IO_ERROR_PRINTLN(...) { DEBUG_PRINTER.println(__VA_ARGS__); }
-  #define IO_ERROR_PRINTBUFFER(buffer, len) { printBuffer(buffer, len); }
-#else
-  #define IO_ERROR_PRINT(...) {}
-  #define IO_ERROR_PRINTLN(...) {}
-  #define IO_ERROR_PRINTBUFFER(buffer, len) {}
-#endif
-
 
 class AdafruitIO {
 

src/AdafruitIO_Definitions.h

@@ -41,18 +41,30 @@ class AdafruitIOGroupCallback {
 
 };
 
+// Uncomment/comment to turn on/off debug output messages.
+#define AIO_DEBUG
 // uncomment/comment to turn on/off error output
 #define AIO_ERROR
 
 // where debug messages will be printed
 #define AIO_PRINTER Serial
+// If using something like Zero or Due, change the above to SerialUSB
+
+// Define actual debug output functions when necessary.
+#ifdef AIO_DEBUG
+  #define AIO_DEBUG_PRINT(...) { AIO_PRINTER.print(__VA_ARGS__); }
+  #define AIO_DEBUG_PRINTLN(...) { AIO_PRINTER.println(__VA_ARGS__); }
+#else
+  #define AIO_DEBUG_PRINT(...) {}
+  #define AIO_DEBUG_PRINTLN(...) {}
+#endif
 
 #ifdef AIO_ERROR
-  #define AIO_ERR_PRINT(...) { AIO_PRINTER.print(__VA_ARGS__); }
-  #define AIO_ERR_PRINTLN(...) { AIO_PRINTER.println(__VA_ARGS__); }
+  #define AIO_ERROR_PRINT(...) { AIO_PRINTER.print(__VA_ARGS__); }
+  #define AIO_ERROR_PRINTLN(...) { AIO_PRINTER.println(__VA_ARGS__); }
 #else
-  #define AIO_ERR_PRINT(...) {}
-  #define AIO_ERR_PRINTLN(...) {}
+  #define AIO_ERROR_PRINT(...) {}
+  #define AIO_ERROR_PRINTLN(...) {}
 #endif
 
 #define AIO_PING_INTERVAL 60000

src/AdafruitIO_Feed.cpp

@@ -152,8 +152,8 @@ AdafruitIO_Data* AdafruitIO_Feed::lastValue()
   url += name;
   url += "/data/retain";
 
-  IO_DEBUG_PRINT("lastValue get ");
-  IO_DEBUG_PRINTLN(url);
+  AIO_DEBUG_PRINT("lastValue get ");
+  AIO_DEBUG_PRINTLN(url);
 
   _io->_http->beginRequest();
   _io->_http->get(url.c_str());
@@ -171,10 +171,10 @@ AdafruitIO_Data* AdafruitIO_Feed::lastValue()
 
   } else {
 
-    IO_ERROR_PRINT("error retrieving lastValue, status: ");
-    IO_ERROR_PRINTLN(status);
-    IO_ERROR_PRINT("response body: ");
-    IO_ERROR_PRINTLN(_io->_http->responseBody());
+    AIO_ERROR_PRINT("error retrieving lastValue, status: ");
+    AIO_ERROR_PRINTLN(status);
+    AIO_ERROR_PRINT("response body: ");
+    AIO_ERROR_PRINTLN(_io->_http->responseBody());
 
     return NULL;