fix

src/Datapoint/ConversionHelpers.cpp

@@ -94,14 +94,14 @@ std::size_t decodeSchedule(const uint8_t* data, std::size_t len, char* output, s
       hour = 0;
       minutes = 0;
     }
-    int result = snprintf(&output[pos], maxLen - pos, "%u.2:%u.2", hour, minutes);
+    int result = snprintf(&output[pos], maxLen - pos, "%.2u:%.2u", hour, minutes);
     if (result < 0) return 0;
     pos += result;
     if (i < 7) {
       output[pos++] = ' ';
     }
   }
-  return pos;
+  return pos + 1;  // include 0-terminator
 }
 
 }  // end namespace VitoWiFi

src/Datapoint/ConversionHelpers.h

@@ -44,10 +44,12 @@ std::size_t encodeSchedule(const char* schedule, uint8_t* output);
 
 /*
 Decodes a byte series to a human-readable schedule consisting of time pairs.
+Returns the number of characters written including 0-terminator
+(will always result in 48 or 0 in case of an error)
 
 Although passed as variables, the function fails when
-- len < 8
-- maxLen < 
+- len != 8
+- maxLen < 48
 */
 std::size_t decodeSchedule(const uint8_t* data, std::size_t len, char* output, std::size_t maxLen);