[vlc-devel] [PATCH 06/21] omxil: Only handle video

[Rémi Denis-Courmont]

On Mon, 5 Sep 2011 12:05:16 +0300 (EEST), Martin Storsjö
<martin at martin.st>
wrote:
> If this workaround was good enough for maemo, perhaps the ifdefs should
be 
> removed completely (sorry for the ugly style of just commenting them out

> in this patch, I can resubmit a new version that removes them instead of

> commenting them out) to use the same workaround for all platforms.

The N900 port never reached "production" or "release" status. So I'm not
sure you should take it as a reference, unless VLC Android is only intended
as a PoC. I don't know what your requirements are, but it should not be
difficult to protect concurrent accesses with a static mutex.

It is entirely possible for VLC to try to spawn two video decoders. There
are video files on the Interwebs (e.g. some videos from the USGS) with
multiple video tracks, and VLC tries to decode all of the by default.

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