Utilize millis() instead of delay for sending payloads. Fixed memory leak issues. Version bump.

Author: epiller

examples/Rapid Development Kit/MonitorPerishableGoods/MonitorPerishableGoods.ino

@@ -55,7 +55,8 @@ Adafruit_BME280 tph1;
 BME280 tph2;
 LED led;
 
-unsigned long previousMillis;
+int publishInterval = 60; // Seconds between data publishing
+unsigned long previousPublishTime;
 float temperature, humidity;
 uint8_t sensorType;
 
@@ -110,24 +111,26 @@ void setSetupError(char* message) {
 
 void loop() {
   att.loop(); // Keep the network and connection to AllThingsTalk alive
-  led.setLight(led.YELLOW);
-  readTphData();
-  delay(1000);
-  led.setLight(led.OFF);
-  payload.reset();
-  payload.set("temp", temperature);
-  payload.set("hum", humidity);
-  debugSerial.print("Temperature: ");
-  debugSerial.println(temperature);
-  debugSerial.print("Humidity: ");
-  debugSerial.println(humidity);
-  led.setLight(led.WHITE);
-  if (att.send(payload)) {
-    led.setLight(led.GREEN, true);
-  } else {
-    led.setLight(led.RED, true);
+  if (millis() - previousPublishTime >= publishInterval*1000) {
+    led.setLight(led.YELLOW);
+    readTphData();
+    delay(1000);
+    led.setLight(led.OFF);
+    payload.reset();
+    payload.set("temp", temperature);
+    payload.set("hum", humidity);
+    debugSerial.print("Temperature: ");
+    debugSerial.println(temperature);
+    debugSerial.print("Humidity: ");
+    debugSerial.println(humidity);
+    led.setLight(led.WHITE);
+    if (att.send(payload)) {
+      led.setLight(led.GREEN, true);
+    } else {
+      led.setLight(led.RED, true);
+    }
+    delay(1000);
+    led.setLight(led.OFF);
+    previousPublishTime = millis();
   }
-  delay(1000);
-  led.setLight(led.OFF);
-  delay(30000);
 }

library.properties

@@ -1,5 +1,5 @@
 name=AllThingsTalk LTE-M SDK
-version=2.0.2
+version=2.0.3
 author=AllThingsTalk <support@allthingstalk.com>
 maintainer=Vanja <vanja@allthingstalk.com>
 sentence=Connect your LTE-M enabled Arduino device to AllThingsTalk IoT Platform.

src/AllThingsTalk_LTEM.cpp

@@ -202,6 +202,7 @@ bool AllThingsTalk_LTEM::send(CborPayload &payload) {
                 debug("> Failed to Publish Message to AllThingsTalk (CBOR)");
                 return false;
             }
+            delete topic;
         }
     } else {
         debug("You're trying to send a message but you've disconnected from the network. Execute connect() to re-connect.");

src/CborPayload.cpp

@@ -16,6 +16,8 @@ CborPayload::~CborPayload() {
 }
 
 void CborPayload::reset() {
+    delete writer;
+    delete output;
     output = new CborStaticOutput(buffer, capacity);
     writer = new CborWriter(*output);
     assetCount = 0;