Merge pull request #1004 from lhag87/lines

change DynamicJsonDocument allocation in my_line_set_points

Author: fvanroie

src/hasp/hasp_attribute.cpp

@@ -197,17 +197,26 @@ static bool my_line_set_points(lv_obj_t* obj, const char* payload)
 {
     my_line_clear_points(obj); // delete pointmap
 
-    // Create new points
-    // Reserve memory for JsonDocument
-    // StaticJsonDocument<1024> doc;
-    size_t maxsize = (128u * ((strlen(payload) / 128) + 1)) + 256;
+    // count LEFT SQUARE BRACKET => nb points +1
+    uint16_t count = 0;
+    for (const char* p = payload; *p; p++) 
+        if (*p == '[') count++;
+    count--;
+    
+    // Reserve memory for JsonDocument rounded to upper 128 bytes
+    uint16_t maxsize = 128u * (3*JSON_ARRAY_SIZE(1)*count / 128+1) ;
+    
+    LOG_VERBOSE(TAG_ATTR,"payload: %s",payload);
+    LOG_TRACE(TAG_ATTR,"count: %u maxsize: %u taille brut %u",count,maxsize,(uint32_t)(3*JSON_ARRAY_SIZE(1)*count));
+
     DynamicJsonDocument doc(maxsize);
     DeserializationError jsonError = deserializeJson(doc, payload);
 
     if(jsonError) { // Couldn't parse incoming JSON payload
         dispatch_json_error(TAG_ATTR, jsonError);
         return false;
     }
+     LOG_VERBOSE(TAG_ATTR,"Memory usage: %u",(uint32_t)doc.memoryUsage());
 
     JsonArray arr  = doc.as<JsonArray>(); // Parse payload
     size_t tot_len = sizeof(lv_point_t*) * (arr.size());