Changeset 64766 in vbox for trunk/src/VBox/Additions/WINNT/VBoxTray

Timestamp:

Nov 30, 2016 10:59:48 AM (8 years ago)

Author:

vboxsync

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

112111

Message:

src/VBox: Make the use of the iterator for RTListForEach()/RTListForEachSafe() more obvious. There is no need to initialize the iterator and we also must not depend on the iterator being NULL if the list was empty.

File:

• trunk/src/VBox/Additions/WINNT/VBoxTray/VBoxLA.cpp (modified) (5 diffs)

trunk/src/VBox/Additions/WINNT/VBoxTray/VBoxLA.cpp

@@ -199 +199 @@
 static void ActionExecutorDeleteActions(RTLISTANCHOR *listActions)
 {
-    ACTIONENTRY *pIter = NULL;
-    ACTIONENTRY *pIterNext = NULL;
+    ACTIONENTRY *pIter;
+    ACTIONENTRY *pIterNext;
     RTListForEachSafe(listActions, pIter, pIterNext, ACTIONENTRY, nodeActionEntry)
     {
@@ -322 +322 @@
         {
             bool fAdded = false;
-            ACTIONENTRY *pIter = NULL;
+            ACTIONENTRY *pIter;
             RTListForEach(listActions, pIter, ACTIONENTRY, nodeActionEntry)
             {
@@ -345 +345 @@
 
 #ifdef LOG_ENABLED
-    ACTIONENTRY *pIter = NULL;
+    ACTIONENTRY *pIter;
     RTListForEach(listActions, pIter, ACTIONENTRY, nodeActionEntry)
     {
@@ -358 +358 @@
     }
 
-    LogFlowFunc(("action enum %d\n",
-           bRet));
+    LogFlowFunc(("action enum %d\n", bRet));
 
     return bRet;
@@ -368 +367 @@
     LogFlowFunc(("ExecuteActions\n"));
 
-    ACTIONENTRY *pIter = NULL;
+    ACTIONENTRY *pIter;
     RTListForEach(listActions, pIter, ACTIONENTRY, nodeActionEntry)
     {