Changeset 76778 in vbox for trunk/src/VBox

Timestamp:

Jan 11, 2019 2:15:04 PM (6 years ago)

Author:

vboxsync

Message:

Audio/DrvAudio: Allow pre-buffering to be disabled by setting it to 0 explicitly.

File:

• trunk/src/VBox/Devices/Audio/DrvAudio.cpp (modified) (2 diffs)

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

@@ -3024 +3024 @@
     if (pDrvCfg->uPreBufSizeMs != UINT32_MAX)
     {
-        if (!pDrvCfg->uPreBufSizeMs) /* Pre-buffering is set to disabled. */
-            LogRel2(("Audio: Using custom pre-buffering (disabled) for stream '%s'\n", pStream->szName));
         pCfgReq->Backend.cfPreBuf = DrvAudioHlpMilliToFrames(pDrvCfg->uPreBufSizeMs, &pCfgReq->Props);
     }
@@ -3100 +3098 @@
                  pStream->szName, DrvAudioHlpFramesToMilli(pCfgAcq->Backend.cfPeriod, &pCfgAcq->Props), pCfgAcq->Backend.cfPeriod));
 
-    if (pCfgAcq->Backend.cfPreBuf != pCfgReq->Backend.cfPreBuf)
+    if (   pCfgReq->Backend.cfPreBuf
+        && pCfgAcq->Backend.cfPreBuf != pCfgReq->Backend.cfPreBuf)
+    {
         LogRel2(("Audio: Pre-buffering size overwritten by backend for stream '%s' (now %RU64ms, %RU32 frames)\n",
                  pStream->szName, DrvAudioHlpFramesToMilli(pCfgAcq->Backend.cfPreBuf, &pCfgAcq->Props), pCfgAcq->Backend.cfPreBuf));
+    }
+    else
+    {
+        LogRel2(("Audio: Pre-buffering is disabled for stream '%s'\n", pStream->szName));
+        pCfgAcq->Backend.cfPreBuf = 0;
+    }
 
     /* Sanity for detecting buggy backends. */