Changeset 73683 in vbox for trunk/src/VBox/Devices/Audio

Timestamp:

Aug 15, 2018 7:15:47 AM (6 years ago)

Author:

vboxsync

Message:

Audio/Mixer: Logging tweaks for audioMixerSinkWriteToStreams().

File:

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

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

@@ -1747 +1747 @@
                                                         &cbChunkWritten);
                 if (RT_FAILURE(rc2))
+                {
+                    if (rc2 == VERR_BUFFER_OVERFLOW)
+                    {
+                        LogRel2(("Mixer: Buffer overrun for mixer stream '%s' (sink '%s')\n", pMixStream->pszName, pSink->pszName));
+#ifdef DEBUG_andy
+                        AssertRC(rc2);
+#endif
+                    }
+                    else if (rc2 != VERR_AUDIO_STREAM_NOT_READY)
+                        LogRel2(("Mixer: Writing to mixer stream '%s' (sink '%s') failed, rc=%Rrc\n", pMixStream->pszName, pSink->pszName, rc2));
+
                     LogFunc(("[%s] Failed writing to stream '%s': %Rrc\n", pSink->pszName, pMixStream->pszName, rc2));
-#ifdef DEBUG_andy
-                AssertRC(rc2);
-#endif
+                }
             }
 
@@ -1767 +1776 @@
         Log3Func(("[%s] cbBuf=%RU32, cbWrittenBuf=%RU32, cbWrittenStream=%RU32\n",
                   pMixStream->pszName, cbBuf, cbWrittenBuf, cbWrittenStream));
-
-        if (cbWrittenBuf < cbBuf)
-        {
-            LogFunc(("[%s] Warning: Only written %RU32/%RU32 bytes for stream '%s'\n",
-                     pSink->pszName, cbWrittenBuf, cbBuf, pMixStream->pszName));
-            LogRel2(("Mixer: Buffer overrun for mixer stream '%s' (sink '%s')\n", pMixStream->pszName, pSink->pszName));
-#ifdef DEBUG_andy
-            AssertFailed();
-#endif
-        }
     }