Small changes for testing ESP8266 AT based devices.

Author: alvarolb

library.properties

@@ -1,5 +1,5 @@
 name=thinger.io
-version=2.6.0
+version=2.6.1
 author=Alvaro Luis Bustamante <[email protected]>
 maintainer=Thinger.io <[email protected]>
 sentence=Arduino library for the Thinger.io Internet of Things Platform.

src/ThingerClient.h

@@ -117,15 +117,20 @@ class ThingerClient : public thinger::thinger {
 
             size_t written = client_.write(temp_data_, out_size_);
             bool success = written == out_size_;
-            free(temp_data_);
-            temp_data_ = NULL;
-            out_size_ = 0;
 
-            #ifdef _DEBUG_
+#ifdef _DEBUG_
             Serial.print(F(" ["));
             Serial.print(success ? F("OK") : F("FAIL"));
             Serial.println(F("]"));
-            #endif
+            if(!success){
+                THINGER_DEBUG_VALUE("THINGER", "Expected:", out_size_);
+                THINGER_DEBUG_VALUE("THINGER", "Wrote:", written);
+            }
+#endif
+
+            free(temp_data_);
+            temp_data_ = NULL;
+            out_size_ = 0;
 
             //FIXME Without this small delay or activating the debug (which takes time), the CC3200 does not work well. Why?
             #ifdef __CC3200R1M1RGC__

src/thinger/thinger.h

@@ -181,9 +181,12 @@ namespace thinger{
             encoder.pb_encode_varint(MESSAGE);
             encoder.pb_encode_varint(sink.bytes_written());
             encoder.encode(message);
-            bool result = write(NULL, 0, true);
-            if(!result) disconnected();
-            return result;
+            return write(NULL, 0, true);
+            /* TODO test this properly. Some devices (like AT based ones)
+             * may fail to write, but are not necessarily disconnected */
+            //bool result = write(NULL, 0, true);
+            //if(!result) disconnected();
+            //return result;
         }
 
         void handle(unsigned long current_time, bool bytes_available)