Make String reallocation warning more descriptive (#8131)

earlephilhower · web-flow · commit f1310c046bcf · 2021-06-17T08:01:32.000-07:00
Fixes #8130
diff --git a/cores/esp8266/WString.cpp b/cores/esp8266/WString.cpp
@@ -182,7 +182,7 @@ bool String::changeBuffer(unsigned int maxStrLen) {
 #ifdef DEBUG_ESP_OOM
     if (!isSSO() && capacity() >= OOM_STRING_THRESHOLD_REALLOC_WARN && maxStrLen > capacity()) {
         // warn when badly re-allocating
-        DEBUGV("[offending String op %d->%d ('%." STR(OOM_STRING_BORDER_DISPLAY) "s ... %." STR(OOM_STRING_BORDER_DISPLAY) "s')]\n",
+        DEBUGV("[String] Reallocating large String(%d -> %d bytes) '%." STR(OOM_STRING_BORDER_DISPLAY) "s ... %." STR(OOM_STRING_BORDER_DISPLAY) "s'\n",
             len(), maxStrLen, c_str(),
             len() > OOM_STRING_BORDER_DISPLAY? c_str() + std::max((int)len() - OOM_STRING_BORDER_DISPLAY, OOM_STRING_BORDER_DISPLAY): "");
     }

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ bool String::changeBuffer(unsigned int maxStrLen) {`
`182`	`182`	`#ifdef DEBUG_ESP_OOM`
`183`	`183`	`if (!isSSO() && capacity() >= OOM_STRING_THRESHOLD_REALLOC_WARN && maxStrLen > capacity()) {`
`184`	`184`	`// warn when badly re-allocating`
`185`		`- DEBUGV("[offending String op %d->%d ('%." STR(OOM_STRING_BORDER_DISPLAY) "s ... %." STR(OOM_STRING_BORDER_DISPLAY) "s')]\n",`
	`185`	`+ DEBUGV("[String] Reallocating large String(%d -> %d bytes) '%." STR(OOM_STRING_BORDER_DISPLAY) "s ... %." STR(OOM_STRING_BORDER_DISPLAY) "s'\n",`
`186`	`186`	`len(), maxStrLen, c_str(),`
`187`	`187`	`len() > OOM_STRING_BORDER_DISPLAY? c_str() + std::max((int)len() - OOM_STRING_BORDER_DISPLAY, OOM_STRING_BORDER_DISPLAY): "");`
`188`	`188`	`}`