Fix replace() failing (#6224)

6591f5bd · s-hadinger · GitHub · 0ea485e5 · 6591f5bd
Unverified Commit 6591f5bd authored Feb 03, 2022 by s-hadinger Committed by GitHub Feb 03, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

cores/esp32/WString.cpp cores/esp32/WString.cpp +2 -1

No files found.
--- a/cores/esp32/WString.cpp
+++ b/cores/esp32/WString.cpp
@@ -774,9 +774,10 @@ void String::replace(const String& find, const String& replace) {
        }
        if(size == len())
            return;
-        if(size > capacity() && !changeBuffer(size))
+        if(size > capacity() && !changeBuffer(size)) {
            log_w("String.Replace() Insufficient space to replace string");
            return;
+        }
        int index = len() - 1;
        while(index >= 0 && (index = lastIndexOf(find, index)) >= 0) {
            readFrom = wbuffer() + index + find.len();