Changeset 14936 in vbox for trunk/src

Timestamp:

Dec 3, 2008 11:07:37 AM (16 years ago)

Author:

vboxsync

Message:

Use CheckComArgStrNotEmptyOrNull where appropriate.

File:

• trunk/src/VBox/Main/VirtualBoxImpl.cpp (modified) (6 diffs)

trunk/src/VBox/Main/VirtualBoxImpl.cpp

@@ -907 +907 @@
                                       IMachine **aMachine)
 {
-    /* null and empty strings are not allowed as path names */
-    if (!aSettingsFile || !(*aSettingsFile))
-        return E_INVALIDARG;
+    CheckComArgStrNotEmptyOrNull(aSettingsFile);
 
     if (!aMachine)
@@ -1197 +1195 @@
                                        IDVDImage2 **aDVDImage)
 {
-    /* null and empty strings are not allowed as path names */
-    if (!aLocation || !(*aLocation))
-        return E_INVALIDARG;
+    CheckComArgStrNotEmptyOrNull(aLocation);
 
     if (!aDVDImage)
@@ -1271 +1267 @@
                                           IFloppyImage2 **aFloppyImage)
 {
-    /* null and empty strings are not allowed as path names */
-    if (!aLocation || !(*aLocation))
-        return E_INVALIDARG;
+    CheckComArgStrNotEmptyOrNull(aLocation);
 
     if (!aFloppyImage)
@@ -2003 +1997 @@
 
 /**
- *  Helper method to that starts a worker thread that:
+ *  Helper method that starts a worker thread that:
  *  - creates a pipe communication channel using SVCHlpClient;
- *  - starts a SVC Helper process that will inherit this channel;
+ *  - starts an SVC Helper process that will inherit this channel;
  *  - executes the supplied function by passing it the created SVCHlpClient
  *    and opened instance to communicate to the Helper process and the given
@@ -2016 +2010 @@
  *  result code returned by the function.
  *
- *  Before the user function is stared, the communication channel passed to in
- *  the \a aClient argument, is fully set up, the function should start using
- *  it's write() and read() methods directly.
+ *  Before the user function is started, the communication channel passed to
+ *  the \a aClient argument is fully set up, the function should start using
+ *  its write() and read() methods directly.
  *
  *  The \a aVrc parameter of the user function may be used to return an error
@@ -2025 +2019 @@
  *  message using this value will be reported to the caller. Note that the
  *  value of \a aVrc is inspected only if the user function itself returns
- *  a success.
+ *  success.
  *
  *  If a failure happens anywhere before the user function would be normally