[vlc-devel] Reworking OpenSLES audio output

Rémi Denis-Courmont remi at remlab.net
Fri May 6 16:36:29 CEST 2011


On Fri, 6 May 2011 15:29:35 +0200, Hugo Beauzee-Luyssen
> New version.
> Best regards.

You've made something simple, a timed loop really complicated. There are
quite many examples how to do that, e.g. ALSA.

Similarly, next_date seems totally useless. That's quite much intricacy to
compute currentDate = mdate().

And please use saner indentation and wrap. It's really annoying to read
with 80 columns.

SLDataFormat_PCM format_pcm could really use named members for
initialization.

Anyway, I still believe the PulseAudio infinite leak bug is present. If
CountAvailableBuffers returns 0, audio buffers are never freed.

-- 
Rémi Denis-Courmont
http://www.remlab.net/



More information about the vlc-devel mailing list