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

Timestamp:

May 31, 2017 1:56:11 PM (8 years ago)

Author:

vboxsync

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

115848

Message:

MediumImpl: @todo

File:

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

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

@@ -6087 +6087 @@
  * @return
  * @note The source format is defined by the Medium instance.
+ *
+ * @todo The only consumer of this method (Appliance::i_writeFSImpl) is already
+ *       on a worker thread, so perhaps consider bypassing the thread here and
+ *       run in the task synchronously?  VBoxSVC has enough threads as it is...
  */
 HRESULT Medium::i_exportFile(const char *aFilename,
@@ -6162 +6166 @@
  * @return
  * @note The destination format is defined by the Medium instance.
+ *
+ * @todo The only consumer of this method (Appliance::i_importOneDiskImage) is
+ *       already on a worker thread, so perhaps consider bypassing the thread
+ *       here and run in the task synchronously?  VBoxSVC has enough threads as
+ *       it is...
  */
 HRESULT Medium::i_importFile(const char *aFilename,