Small fixes

companion/src/firmwares/edgetx/edgetxinterface.cpp

@@ -38,7 +38,7 @@ static void writeYamlToByteArray(const YAML::Node& node, QByteArray& data)
     data_ostream << node;
     data = QByteArray::fromStdString(data_ostream.str());
 
-    qDebug() << "Saving radio.yml:";
+    qDebug() << "Saving YAML:";
     qDebug() << data_ostream.str().c_str();
 }
 
@@ -139,5 +139,5 @@ bool writeRadioSettingsToYaml(const GeneralSettings& settings, QByteArray& data)
   node = settings;
 
   writeYamlToByteArray(node, data);
-  return false;
+  return true;
 }

companion/src/firmwares/edgetx/yaml_modeldata.cpp

@@ -50,7 +50,7 @@ Node convert<TimerData>::encode(const TimerData& rhs)
   node["swtch"] = rhs.swtch;
   node["mode"] = timerModeLut << rhs.mode;
   node["name"] = rhs.name;
-  node["minuteBeep"] = rhs.minuteBeep;
+  node["minuteBeep"] = (int)rhs.minuteBeep;
   node["countdownBeep"] = rhs.countdownBeep;
   node["start"] = rhs.val;
   node["persistent"] = rhs.persistent;
@@ -86,15 +86,15 @@ Node convert<ModelData>::encode(const ModelData& rhs)
     node["timers"][std::to_string(i)] = timer;
   }
 
-  node["noGlobalFunctions"] = rhs.noGlobalFunctions;
-  node["thrTrim"] = rhs.thrTrim;
+  node["noGlobalFunctions"] = (int)rhs.noGlobalFunctions;
+  node["thrTrim"] = (int)rhs.thrTrim;
   node["trimInc"] = rhs.trimInc;
   node["displayTrims"] = rhs.trimsDisplay;
-  node["disableThrottleWarning"] = rhs.disableThrottleWarning;
+  node["disableThrottleWarning"] = (int)rhs.disableThrottleWarning;
   // node[] = rhs.beepANACenter;
-  node["extendedLimits"] = rhs.extendedLimits;
-  node["extendedTrims"] = rhs.extendedTrims;
-  node["throttleReversed"] = rhs.throttleReversed;
+  node["extendedLimits"] = (int)rhs.extendedLimits;
+  node["extendedTrims"] = (int)rhs.extendedTrims;
+  node["throttleReversed"] = (int)rhs.throttleReversed;
 
   // flightModeData[]
   // mixData[]
@@ -113,7 +113,7 @@ Node convert<ModelData>::encode(const ModelData& rhs)
   // potsWarnEnabled[]
   // potsWarnPosition[]
 
-  node["displayChecklist"] = rhs.displayChecklist;
+  node["displayChecklist"] = (int)rhs.displayChecklist;
 
   // gvarData[]
   // mavlink ???

companion/src/firmwares/edgetx/yaml_rawswitch.cpp

@@ -139,7 +139,7 @@ bool convert<RawSwitch>::decode(const Node& node, RawSwitch& rhs)
     // TODO: check if we have a match?
     //       or does RawSwitch default to NONE anyway?
     //
-    node >> switchTypeLut >> rhs.type;
+    node >> rhs.type;
     rhs.index = 0;
   }
 

companion/src/firmwares/edgetx/yaml_switchconfig.h

@@ -55,7 +55,7 @@ struct YamlKnobConfig : public YamlNameConfig<N, name_lut> {
                  const unsigned int in_configs[N]) :
       YamlNameConfig<N, name_lut>(in_names)
   {
-    memset(config, 0, sizeof(config));
+    memcpy(config, in_configs, sizeof(config));
   }
 
   void copy(char out_names[N][HARDWARE_NAME_LEN + 1],

companion/src/storage/categorized.cpp

@@ -311,9 +311,10 @@ bool CategorizedStorageFormat::writeYaml(const RadioData & radioData)
   // TODO: sort 'modelFiles' by category index
   QByteArray modelslistBuffer;
   if (!writeModelsListToYaml(radioData.categories, modelFiles, modelslistBuffer)
-      || writeFile(modelslistBuffer, "MODELS/models.yml")) {
+      || !writeFile(modelslistBuffer, "MODELS/models.yml")) {
     return false;
   }
 
-  return true;
+  // force error to avoid overwriting the archive...
+  return false;
 }