Changeset 78934 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

Jun 3, 2019 12:47:42 PM (6 years ago)

Author:

vboxsync

Message:

FE/Qt: bugref:9434: Import Appliance wizard: Form Editor widget: A bit of cleanup.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/wizards/importappliance/UIFormEditorWidget.cpp (modified) (3 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/wizards/importappliance/UIFormEditorWidget.cpp

@@ -58 +58 @@
 
     /** Constructs null choice data. */
-    ChoiceData() {}
+    ChoiceData()
+        : m_iSelectedIndex(-1) {}
     /** Constructs choice data on the basis of passed @a values and @a iSelectedIndex. */
     ChoiceData(const QVector<QString> &values, int iSelectedIndex)
@@ -101 +102 @@
 
     /** Constructs null ranged-integer data. */
-    RangedIntegerData() {}
+    RangedIntegerData()
+        : m_iMinimum(-1), m_iMaximum(-1), m_iInteger(-1) {}
     /** Constructs ranged-integer data on the basis of passed @a iMinimum, @a iMaximum and @a iInteger. */
     RangedIntegerData(int iMinimum, int iMaximum, int iInteger)
@@ -804 +806 @@
                     {
                         case KFormValueType_String:
+                        {
                             m_dataList[index.row()]->setString(value.toString());
                             emit dataChanged(index, index);
                             updateGeneration();
                             return true;
+                        }
                         case KFormValueType_Choice:
+                        {
                             m_dataList[index.row()]->setChoice(value.value<ChoiceData>());
                             emit dataChanged(index, index);
                             updateGeneration();
                             return true;
+                        }
                         case KFormValueType_RangedInteger:
+                        {
                             m_dataList[index.row()]->setRangedInteger(value.value<RangedIntegerData>());
                             emit dataChanged(index, index);
                             updateGeneration();
                             return true;
+                        }
                         default:
                             return false;