[MPlayer-cvslog] r34341 - trunk/libmpdemux/demux_lavf.c

[Reimar Döffinger]

On Thu, Nov 10, 2011 at 05:31:08PM +0100, cehoyos wrote:
> Author: cehoyos
> Date: Thu Nov 10 17:31:07 2011
> New Revision: 34341
> 
> Log:
> Fix .qcp (raw QCELP) playback.
> 
> Modified:
>    trunk/libmpdemux/demux_lavf.c
> 
> Modified: trunk/libmpdemux/demux_lavf.c
> ==============================================================================
> --- trunk/libmpdemux/demux_lavf.c	Thu Nov 10 17:06:23 2011	(r34340)
> +++ trunk/libmpdemux/demux_lavf.c	Thu Nov 10 17:31:07 2011	(r34341)
> @@ -220,6 +220,7 @@ static const char * const preferred_list
>      "mxf",
>      "ogg",
>      "pva",
> +    "qcp",

This should never be necessary to fix playback, and except
for corner cases we really should try to fix it properly.
So can you point to a sample?
In addition I'd like to point out that my personal process
to add to this list involves verify that the probe function in
lavf
a) does not need a huge amount of data
b) checks at least the equivalent of 64 bit.
(qcp demuxer does fulfill those requirements though, so
no complaint there).