<html>
  <head>
    <meta content="text/html; charset=ISO-8859-15"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 07/27/2011 09:13 PM, Rémi Denis-Courmont wrote:
    <blockquote cite="mid:201107272213.57844.remi@remlab.net"
      type="cite">
      <pre wrap="">Le mercredi 27 juillet 2011 21:53:21 Hans-Kristian Arntzen, vous avez écrit :
</pre>
      <blockquote type="cite">
        <pre wrap="">Found an issue with the OSS driver, hopefully changes are accepted to it
since it seemed to be really old.
</pre>
      </blockquote>
      <pre wrap="">
Errm, does GETODELAY really return a byte count?
This would be rather counter-intuitive...

On the cosmetic side, this would avoid duplicating the ifdef:

+#ifdef SNDCTL_DSP_GETODELAY
+    if ( ioctl( p_sys->i_fd, SNDCTL_DSP_GETODELAY, &i_bytes ) < 0 )
+#endif
     {

</pre>
    </blockquote>
    Yes, GETODELAY returns latency in bytes. From OSSv4 documentation:
    <a class="moz-txt-link-freetext" href="http://manuals.opensound.com/developer/SNDCTL_DSP_GETODELAY.html">http://manuals.opensound.com/developer/SNDCTL_DSP_GETODELAY.html</a><br>
    <br>
    "This ioctl call tells how long time it's going to take before the
    next sample to be written gets played by the hardware. The delay is
    returned in bytes and it can be converted to seconds by dividing it
    by <code>sample_rate*num_of_channels*sample_size_in_bytes</code>."<br>
    <br>
    Updated patch to avoid #ifdef duplication. <br>
    <br>
  </body>
</html>