Changeset 96172 in vbox for trunk/src/VBox/Main/xml

Timestamp:

Aug 12, 2022 1:27:46 PM (2 years ago)

Author:

vboxsync

Message:

Recording/Main: Changed IRecordingScreenSettings::features attribute to get/set a safe array of RecordingFeature entries instead of an anonymous ULONG. Makes it much easier to use. bugref:10275

File:

• trunk/src/VBox/Main/xml/Settings.cpp (modified) (3 diffs)

trunk/src/VBox/Main/xml/Settings.cpp

@@ -2807 +2807 @@
     for (size_t i = 0; i < lstFeatures.size(); i++)
     {
-        if (lstFeatures.at(i).compare("video", RTCString::CaseInsensitive))
+        if (lstFeatures.at(i).compare("video", RTCString::CaseInsensitive) == 0)
             featureMap[RecordingFeature_Video] = true;
-        else if (lstFeatures.at(i).compare("audio", RTCString::CaseInsensitive))
+        else if (lstFeatures.at(i).compare("audio", RTCString::CaseInsensitive) == 0)
             featureMap[RecordingFeature_Audio] = true;
         /* ignore everything else */
@@ -2826 +2826 @@
 void RecordingScreenSettings::featuresToString(const RecordingFeatureMap &featureMap, com::Utf8Str &strFeatures)
 {
+    strFeatures = "";
+
     RecordingFeatureMap::const_iterator itFeature = featureMap.begin();
     while (itFeature != featureMap.end())
@@ -2833 +2835 @@
         if (itFeature->first == RecordingFeature_Audio && itFeature->second)
             strFeatures += "audio ";
-        strFeatures += " ";
-
         ++itFeature;
     }