Changeset 89518 in vbox for trunk/src/VBox/Devices

Timestamp:

Jun 4, 2021 10:34:56 PM (4 years ago)

Author:

vboxsync

Message:

DrvHostAudioOss: Comment updates. bugref:9890

File:

• trunk/src/VBox/Devices/Audio/DrvHostAudioOss.cpp (modified) (4 diffs)

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

@@ -688 +688 @@
 
     /*
-     * Note! This logic was found in StreamPlay and corrected a little.
+     * The logic here must match what StreamPlay does.
      *
-     * The logic here must match what StreamPlay does.
+     * Note! We now use 'bytes' rather than the fragments * fragsize as we used
+     *       to do (up to 2021), as these are documented as obsolete.
      */
     audio_buf_info BufInfo = { 0, 0, 0, 0 };
@@ -740 +741 @@
 
     /*
-     * Figure out now much to write (same as drvHstAudOssHA_StreamGetWritable).
+     * Figure out now much to write (same as drvHstAudOssHA_StreamGetWritable,
+     * must match exactly).
      */
     audio_buf_info BufInfo;
@@ -815 +817 @@
      * Use SNDCTL_DSP_GETISPACE to see how much we can read.
      *
-     * Note! We use bytes rather than the fragments * fragsize, as these are
-     *       documented as obsolete.  (Playback code should do the same.)
+     * Note! We now use 'bytes' rather than the fragments * fragsize as we used
+     *       to do (up to 2021), as these are documented as obsolete.
      */
     audio_buf_info BufInfo = { 0, 0, 0, 0 };
@@ -846 +848 @@
         cbRet = PDMAudioPropsFramesToBytes(&pStreamOSS->Cfg.Props, 1);
 
-    Log4Func(("returns %#x (%u) [cbBuf=%#x)\n", cbRet, cbRet, cbBuf));
+    Log4Func(("returns %#x (%u) [cbBuf=%#x]\n", cbRet, cbRet, cbBuf));
     return cbRet;
 }