Changeset 43827 in vbox

Timestamp:

Nov 7, 2012 11:02:12 AM (12 years ago)

Author:

vboxsync

Message:

FE/Qt: VM group UI: Some cleanup for group-item (root-item processing related).

File:

• trunk/src/VBox/Frontends/VirtualBox/src/selector/graphics/chooser/UIGChooserItemGroup.cpp (modified) (10 diffs)

trunk/src/VBox/Frontends/VirtualBox/src/selector/graphics/chooser/UIGChooserItemGroup.cpp

@@ -173 +173 @@
 QString UIGChooserItemGroup::fullName() const
 {
-    /* Return "/" for root-group: */
-    if (!parentItem())
+    /* Return "/" for main root-item: */
+    if (isMainRoot())
         return "/";
     /* Get full parent name, append with '/' if not yet appended: */
+    AssertMsg(parentItem(), ("Incorrect parent set!"));
     QString strFullParentName = parentItem()->fullName();
     if (!strFullParentName.endsWith('/'))
@@ -216 +217 @@
 void UIGChooserItemGroup::close(bool fAnimated /* = true */)
 {
-    AssertMsg(parentItem(), ("Can't close root-item!"));
+    AssertMsg(!isRoot(), ("Can't close root-item!"));
     m_pToggleButton->setToggled(false, fAnimated);
 }
@@ -222 +223 @@
 void UIGChooserItemGroup::open(bool fAnimated /* = true */)
 {
-    AssertMsg(parentItem(), ("Can't open root-item!"));
+    AssertMsg(!isRoot(), ("Can't open root-item!"));
     m_pToggleButton->setToggled(true, fAnimated);
 }
@@ -232 +233 @@
         if (pItem->toMachineItem()->id() == strId)
             return true;
+    /* Found nothing? */
     return false;
 }
@@ -245 +247 @@
         if (pItem->toGroupItem()->isContainsLockedMachine())
             return true;
+    /* Found nothing? */
     return false;
 }
@@ -263 +266 @@
 void UIGChooserItemGroup::sltNameEditingFinished()
 {
-    /* Not for root-item: */
+    /* Not for root: */
     if (isRoot())
         return;
@@ -290 +293 @@
 void UIGChooserItemGroup::sltGroupToggleStart()
 {
-    /* Not for root-item: */
+    /* Not for root: */
     if (isRoot())
         return;
@@ -321 +324 @@
 void UIGChooserItemGroup::sltGroupToggleFinish(bool fToggled)
 {
-    /* Not for root-item: */
+    /* Not for root: */
     if (isRoot())
         return;
@@ -693 +696 @@
 void UIGChooserItemGroup::startEditing()
 {
-    /* Not for root-item: */
+    /* Not for root: */
     if (isRoot())
         return;
@@ -892 +895 @@
             return pFoundItem;
 
-    /* Nothing found? */
+    /* Found nothing? */
     return 0;
 }