Merge pull request letscontrolit#3946 from TD-er/bugfix/RAMtracker

[RAM tracker] fix potential multiple calls to std::move on the same string

Author: TD-er

src/src/Globals/RamTracker.cpp

@@ -51,18 +51,23 @@ void checkRAM(const __FlashStringHelper * descr ) {
 void checkRAM(String&& descr ) {
   if (Settings.EnableRAMTracking())
     myRamTracker.registerRamState(descr);
-  
-  uint32_t freeRAM = FreeMem();
+
+  const uint32_t freeStack = getFreeStackWatermark();
+  if (freeStack <= lowestFreeStack) {
+    lowestFreeStack = freeStack;
+    lowestFreeStackfunction = descr;
+  }
+
+#ifdef ESP32
+  const uint32_t freeRAM = ESP.getMinFreeHeap();
+#else
+  const uint32_t freeRAM = FreeMem();
+#endif
   if (freeRAM <= lowestRAM)
   {
     lowestRAM = freeRAM;
     lowestRAMfunction = std::move(descr);
   }
-  uint32_t freeStack = getFreeStackWatermark();
-  if (freeStack <= lowestFreeStack) {
-    lowestFreeStack = freeStack;
-    lowestFreeStackfunction = std::move(descr);
-  }
 }