Fix default model index

radio/src/model_init.h

@@ -24,5 +24,11 @@
 void clearInputs();
 void setDefaultInputs();
 
+#if defined(STORAGE_MODELSLIST)
+  #define DEFAULT_MODEL_IDX 1
+#else
+  #define DEFAULT_MODEL_IDX 0
+#endif
+
 // Reset the global model to full defaults
-void setModelDefaults(uint8_t id);
+void setModelDefaults(uint8_t id = DEFAULT_MODEL_IDX);

radio/src/storage/eeprom_common.cpp

@@ -102,7 +102,7 @@ void storageEraseAll(bool warn)
   TRACE("storageEraseAll");
 
   generalDefault();
-  setModelDefaults(0);
+  setModelDefaults();
 
   if (warn) {
     ALERT(STR_STORAGE_WARNING, STR_BAD_RADIO_DATA, AU_BAD_RADIODATA);

radio/src/storage/sdcard_common.cpp

@@ -69,11 +69,9 @@ void storageFormat()
   sdCheckAndCreateDirectory(RADIO_PATH);
   sdCheckAndCreateDirectory(MODELS_PATH);
   generalDefault();
+  setModelDefaults();
 #if defined(STORAGE_MODELSLIST)
-  setModelDefaults(1);
   storageCreateModelsList();
-#else
-  setModelDefaults(0);
 #endif
 }
 

radio/src/tests/gtests.h

@@ -106,7 +106,7 @@ class OpenTxTest : public testing::Test
       SYSTEM_RESET();
       MODEL_RESET();
       MIXER_RESET();
-      setModelDefaults(0);
+      setModelDefaults();
       RADIO_RESET();
     }
 };

radio/src/tests/mixer.cpp

@@ -467,7 +467,7 @@ TEST(Curves, LinearIntpol)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   for (int8_t i=-2; i<=2; i++) {
     g_model.points[2+i] = 50*i;
   }
@@ -673,7 +673,7 @@ TEST_F(TrimsTest, throttleTrimEle) {
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_eeGeneral.templateSetup = 17;
   applyDefaultTemplate();
   g_model.thrTrim = 1;
@@ -686,7 +686,7 @@ TEST_F(TrimsTest, throttleTrimEle) {
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_eeGeneral.templateSetup = 0;
   applyDefaultTemplate();
 }
@@ -697,7 +697,7 @@ TEST(Heli, BasicTest)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_model.swashR.collectiveSource = MIXSRC_Thr;
   g_model.swashR.elevatorSource = MIXSRC_Ele;
   g_model.swashR.aileronSource = MIXSRC_Ail;
@@ -729,7 +729,7 @@ TEST(Heli, Mode2Test)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_eeGeneral.templateSetup = 2;
   applyDefaultTemplate();
   g_model.swashR.collectiveSource = MIXSRC_Thr;
@@ -765,7 +765,7 @@ TEST(Trainer, UnpluggedTest)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_model.mixData[0].destCh = 0;
   g_model.mixData[0].mltpx = MLTPX_ADD;
   g_model.mixData[0].srcRaw = MIXSRC_FIRST_TRAINER;
@@ -782,7 +782,7 @@ TEST_F(MixerTest, flightModeTransition)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_model.flightModeData[1].swtch = TR(SWSRC_ID2, SWSRC_SA2);
   g_model.flightModeData[0].fadeIn = 100;
   g_model.flightModeData[0].fadeOut = 100;
@@ -808,7 +808,7 @@ TEST_F(MixerTest, flightModeOverflow)
   SYSTEM_RESET();
   MODEL_RESET();
   MIXER_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   g_model.flightModeData[1].swtch = TR(SWSRC_ID2, SWSRC_SA2);
   g_model.flightModeData[0].fadeIn = 100;
   g_model.flightModeData[0].fadeOut = 100;

radio/src/tests/switches.cpp

@@ -78,7 +78,7 @@ TEST(getSwitch, nullSW)
 TEST(getSwitch, inputWithTrim)
 {
   MODEL_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   MIXER_RESET();
 
   // g_model.logicalSw[0] = { LS_FUNC_VPOS, MIXSRC_FIRST_INPUT, 0, 0 };
@@ -100,7 +100,7 @@ TEST(evalLogicalSwitches, playFile)
 {
   SYSTEM_RESET();
   MODEL_RESET();
-  setModelDefaults(0);
+  setModelDefaults();
   MIXER_RESET();
 
   extern BitField<(MAX_LOGICAL_SWITCHES * 2/*on, off*/)> sdAvailableLogicalSwitchAudioFiles;