Changeset 92196 in vbox

Timestamp:

Nov 3, 2021 5:33:11 PM (3 years ago)

Author:

vboxsync

Message:

Audio/Validation Kit: More code for audio data beacon handling. Now has dedicated beacons for recording / playback tests [build fix]. bugref:10008

Location:

trunk/src/VBox/Devices/Audio

Files:

• AudioTest.cpp (modified) (1 diff)
• DrvHostAudioValidationKit.cpp (modified) (1 diff)

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

@@ -2473 +2473 @@
             && pauBuf[i + 3] == byBeacon)
         {
-            //if (pBeacon->cbProcessed)
-            //{
-                //LogRel(("AudioTestBeaconAddConsecutive: Beacon %s started (last was %RU32@%zu)\n", AudioTestBeaconTypeGetName(pBeacon->enmType), cbBeacon, i));
-                if (offGap)
-                {
-                    pBeacon->cbProcessed = 0;
-                }
-                pBeacon->cbProcessed += cbStep;
-                offGap = 0;
-                //fInBeacon = true;
-                //offLastBeacon = i;
-            //}
-            //cbBeacon += cbFrameSize;
+            if (offGap)
+            {
+                pBeacon->cbProcessed = 0;
+            }
+            pBeacon->cbProcessed += cbStep;
+            offGap = 0;
         }
         else
-        {
-        #if 0
-            if (fInBeacon)
-            {
-                //LogRel(("AudioTestBeaconAddConsecutive: Beacon %s ended (%RU32@%zu)\n", AudioTestBeaconTypeGetName(pBeacon->enmType), cbBeacon, offLastBeacon));
-                fInBeacon = false;
-                continue;
-            }
-        #endif
             offGap  = i;
-        }
-    }
-
-#if 0
-    if (   cbBeacon
-        && (   fInBeacon
-            || offLastBeacon == 0)
-       )
-    {
-        pBeacon->cbProcessed += cbBeacon;
-        Assert(pBeacon->cbProcessed <= pBeacon->cbToProcess);
-    }
-
-    if (cbBeacon)
-        LogRel(("AudioTestBeaconAddConsecutive: %s in=%RTbool cb=%RU32 -> cbProc=%RU32\n", AudioTestBeaconTypeGetName(pBeacon->enmType), fInBeacon, cbBeacon, pBeacon->cbProcessed));
-//LogRel(("AudioTestBeaconAddConsecutive: Beacon proc finished (last was cbBuf=%RU32, fInBeacon=%RTbool, cbBeacon=%RU32, cbProc=%RU32)\n", cbBuf, fInBeacon, cbBeacon, pBeacon->cbProcessed));
-#endif
+    }
 
     Assert(pBeacon->cbProcessed >= cbProcessedInitial);

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

@@ -1112 +1112 @@
                 /* Whether we count all silence as recorded data or not.
                  * Currently we don't, as otherwise consequtively played tones will be cut off in the end. */
-                bool const fCountAllSilenceAsData = false;
-                if (   !fIsAllSilence
-                    || (fIsAllSilence == fCountAllSilenceAsData))
+                if (!fIsAllSilence)
                     pTst->t.TestTone.u.Play.cbRead += cbBuf;