Changeset 60045 in vbox for trunk/src/VBox/Main

Timestamp:

Mar 15, 2016 8:37:54 PM (9 years ago)

Author:

vboxsync

Message:

Main/ConsoleImpl2: warnings (found by PVS)

File:

• trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp (modified) (4 diffs)

trunk/src/VBox/Main/src-client/ConsoleImpl2.cpp

@@ -2532 +2532 @@
             BOOL fEnabledSerPort = FALSE;
             if (serialPort)
+            {
                 hrc = serialPort->COMGETTER(Enabled)(&fEnabledSerPort);                     H();
+            }
             if (!fEnabledSerPort)
                 continue;
@@ -2669 +2671 @@
         hrc = pMachine->COMGETTER(AudioAdapter)(audioAdapter.asOutParam());                 H();
         if (audioAdapter)
+        {
             hrc = audioAdapter->COMGETTER(Enabled)(&fAudioEnabled);                         H();
+        }
 
         if (fAudioEnabled)
@@ -4706 +4710 @@
                     pos = ppos = 0;
 #define ITERATE_TO_NEXT_TERM(res, str, pos, ppos) \
-    do { \
+    { \
         pos = str.find(",", ppos); \
         if (pos == Utf8Str::npos) \
@@ -4716 +4720 @@
         Log2((#res " %s pos:%d, ppos:%d\n", res.c_str(), pos, ppos)); \
         ppos = pos + 1; \
-    } while (0)
+    } /* no do { ... } while because of 'continue' */
                     ITERATE_TO_NEXT_TERM(strName, utf, pos, ppos);
                     ITERATE_TO_NEXT_TERM(strProto, utf, pos, ppos);