Changeset 63171 in vbox for trunk/src/VBox

Timestamp:

Aug 8, 2016 2:40:45 PM (8 years ago)

Author:

vboxsync

Message:

Main: warning about switches

File:

• trunk/src/VBox/Main/src-server/ApplianceImplImport.cpp (modified) (2 diffs)

trunk/src/VBox/Main/src-server/ApplianceImplImport.cpp

@@ -3807 +3807 @@
 
             // find the OVF virtual system description entry for this storage controller
+/** @todo
+ * r=bird: What on earh this is switch supposed to do?  (I've added the default:break;, so don't
+ * get confused by it.)  Kind of looks like it's supposed to do something error handling related
+ * in the default case...
+ */
             switch (sc.storageBus)
             {
@@ -3817 +3822 @@
                 case StorageBus_SAS:
                     break;
+                default: break; /* Shut up MSC. */
             }