Changeset 60990 in vbox

Timestamp:

May 15, 2016 5:58:13 PM (9 years ago)

Author:

vboxsync

Message:

DrvAudio.cpp: Replaced the generic VERR_NO_MORE_HANDLES with a dedicated one (VERR_AUDIO_NO_FREE_INPUT_STREAMS). This error occurs here with a default win8.1 config.

Location:

trunk

Files:

• include/VBox/err.h (modified) (1 diff)
• src/VBox/Devices/Audio/DrvAudio.cpp (modified) (2 diffs)

trunk/include/VBox/err.h

@@ -2743 +2743 @@
  *  NULL audio driver. */
 #define VERR_AUDIO_BACKEND_INIT_FAILED              (-6600)
+/** No free input streams.  */
+#define VERR_AUDIO_NO_FREE_INPUT_STREAMS            (-6601)
 /** @} */
 

trunk/src/VBox/Devices/Audio/DrvAudio.cpp

@@ -845 +845 @@
     {
         LogFlowFunc(("No more input streams free to use, bailing out\n"));
-        return VERR_NO_MORE_HANDLES;
+        return VERR_AUDIO_NO_FREE_INPUT_STREAMS;
     }
 
@@ -866 +866 @@
     int rc;
 
-    do
+    do /* goto avoidance */
     {
         uint32_t cSamples = 0;
         rc = pThis->pHostDrvAudio->pfnInitIn
            ? pThis->pHostDrvAudio->pfnInitIn(pThis->pHostDrvAudio, pHstStrmIn,
-                                             pCfg, pCfg->DestSource.Source, &cSamples) : VINF_SUCCESS;
+                                             pCfg, pCfg->DestSource.Source, &cSamples)
+           : VINF_SUCCESS;
         if (RT_FAILURE(rc))
         {