Changeset 24137 in vbox for trunk/src/VBox/Frontends/VBoxManage

Timestamp:

Oct 28, 2009 1:16:17 PM (16 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

54057

Message:

VBoxManageModifyVM: no need for ValueUnion.u32/64 zero checks, its main's job

File:

• trunk/src/VBox/Frontends/VBoxManage/VBoxManageModifyVM.cpp (modified) (9 diffs)

trunk/src/VBox/Frontends/VBoxManage/VBoxManageModifyVM.cpp

@@ -284 +284 @@
             case MODIFYVM_MEMORY:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(MemorySize)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(MemorySize)(ValueUnion.u32));
                 break;
             }
@@ -291 +290 @@
             case MODIFYVM_VRAM:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(VRAMSize)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(VRAMSize)(ValueUnion.u32));
                 break;
             }
@@ -460 +458 @@
             case MODIFYVM_CPUS:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(CPUCount)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(CPUCount)(ValueUnion.u32));
                 break;
             }
@@ -467 +464 @@
             case MODIFYVM_MONITORCOUNT:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(MonitorCount)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(MonitorCount)(ValueUnion.u32));
                 break;
             }
@@ -549 +545 @@
             case MODIFYVM_BIOSLOGODISPLAYTIME:
             {
-                if (ValueUnion.u64 > 0)
-                    CHECK_ERROR(biosSettings, COMSETTER(LogoDisplayTime)(ValueUnion.u64));
+                CHECK_ERROR(biosSettings, COMSETTER(LogoDisplayTime)(ValueUnion.u64));
                 break;
             }
@@ -584 +579 @@
             case MODIFYVM_BIOSSYSTEMTIMEOFFSET:
             {
-                if (ValueUnion.u64 > 0)
-                    CHECK_ERROR(biosSettings, COMSETTER(TimeOffset)(ValueUnion.u64));
+                CHECK_ERROR(biosSettings, COMSETTER(TimeOffset)(ValueUnion.u64));
                 break;
             }
@@ -1489 +1483 @@
             case MODIFYVM_GUESTSTATISTICSINTERVAL:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(StatisticsUpdateInterval)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(StatisticsUpdateInterval)(ValueUnion.u32));
                 break;
             }
@@ -1497 +1490 @@
             case MODIFYVM_GUESTMEMORYBALLOON:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(MemoryBalloonSize)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(MemoryBalloonSize)(ValueUnion.u32));
                 break;
             }
@@ -1801 +1793 @@
             case MODIFYVM_TELEPORTER_PORT:
             {
-                if (ValueUnion.u32 > 0)
-                    CHECK_ERROR(machine, COMSETTER(TeleporterPort)(ValueUnion.u32));
+                CHECK_ERROR(machine, COMSETTER(TeleporterPort)(ValueUnion.u32));
                 break;
             }