Changeset 68719 in vbox

Timestamp:

Sep 12, 2017 9:48:11 AM (7 years ago)

Author:

vboxsync

Message:

Audio: TMTimerStop() not needed in hdaTimerStop() / ichac97TimerStop().

Location:

trunk/src/VBox/Devices/Audio

Files:

• DevHDA.cpp (modified) (1 diff)
• DevIchAc97.cpp (modified) (1 diff)

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

@@ -2561 +2561 @@
         pThis->fTimerActive = false;
 
-        TMTimerStop(pThis->pTimer);
+        /* Note: Do not stop the timer via TMTimerStop() here, as there still might
+         *       be queued audio data which needs to be handled (e.g. played back) first
+         *       before actually stopping the timer for good. */
     }
 

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

@@ -2151 +2151 @@
         pThis->fTimerActive = false;
 
-        TMTimerStop(pThis->pTimer);
+        /* Note: Do not stop the timer via TMTimerStop() here, as there still might
+         *       be queued audio data which needs to be handled (e.g. played back) first
+         *       before actually stopping the timer for good. */
     }