Complete TODO removes

Author: nseidle

Firmware/RTK_Everywhere/GNSS_UM980.ino

@@ -252,12 +252,6 @@ bool GNSS_UM980::configureOnce()
     response &= _um980->setPortBaudrate("COM2", 115200); // COM2 is connected to the IMU
     response &= _um980->setPortBaudrate("COM3", 115200); // COM3 is connected to the switch, then ESP32
 
-    // TODO: delete Paul's little rant, once you've read it
-    // Why would you change the COM3 baud rate here? It's connected to the ESP32 serialGNSS. You can't
-    // change the COM3 baud rate without changing the serialGNSS baud to match... I think this only works
-    // because there is a "settings.dataPortBaud = 115200; // Override settings. Use UM980 at 115200bps."
-    // in beginBoard. It causes badness when profile defaults are applied via web config, because that
-    // sets settings.dataPortBaud back to its default of 230400!
     //  // For now, let's not change the baud rate of the interface. We'll be using the default 115200 for now.
     //  response &= setBaudRateCOM3(settings.dataPortBaud); // COM3 is connected to ESP UART2
 

Firmware/RTK_Everywhere/menuCommands.ino

@@ -2016,11 +2016,6 @@ void createSettingsString(char *newSettings)
     }
     stringRecord(newSettings, "lastState", lastState);
 
-    /* This is a duplicate - from olden times? See #719 It's probably not doing any harm. But still... TODO: remove it
-    stringRecord(
-        newSettings, "profileName",
-        profileNames[profileNumber]); // Must come before profile number so AP config page JS has name before number
-    */
     stringRecord(newSettings, "profileNumber", profileNumber);
     for (int index = 0; index < MAX_PROFILE_COUNT; index++)
     {