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

[Reimar Döffinger]

On Thu, Nov 10, 2011 at 09:57:36PM +0000, Carl Eugen Hoyos wrote:
> Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:
> 
> > > +    "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?
> 
> fate-suite/qcp
> 
> Please explain how this should be done properly, thank you, Carl Eugen

I won't know before debugging why it failed before.
I'd presume one of our native demuxers grabs it incorrectly.
In that case it should be improved to not do that if easily possible.
If not we might just leave it as-is though, since as said the probe
function is quite fine.