Changeset 40492 in vbox for trunk/src/VBox/Main/src-server

Timestamp:

Mar 15, 2012 8:53:37 PM (13 years ago)

Author:

vboxsync

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

76870

Message:

Main/Machine+Snapshot: add a comment to make it easier to get rid of those time-consuming commits on startup one day

Location:

trunk/src/VBox/Main/src-server

Files:

• MachineImpl.cpp (modified) (1 diff)
• SnapshotImpl.cpp (modified) (1 diff)

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

@@ -723 +723 @@
         /* commit all changes made during loading the settings file */
         commit(); // @todo r=dj why do we need a commit during init?!? this is very expensive
+        /// @todo r=klaus for some reason the settings loading logic backs up
+        // the settings, and therefore a commit is needed. Should probably be changed.
     }
     else

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

@@ -1165 +1165 @@
     if (SUCCEEDED(rc))
         /* commit all changes made during the initialization */
-        commit();   // @todo r=dj why do we need a commit in init?!? this is very expensive
+        commit();   /// @todo r=dj why do we need a commit in init?!? this is very expensive
+        /// @todo r=klaus for some reason the settings loading logic backs up
+        // the settings, and therefore a commit is needed. Should probably be changed.
 
     /* Confirm a successful initialization when it's the case */