[MPlayer-dev-eng] Re: [PATCH] don't skip first mp3 frame

[Reimar Döffinger]

Hi,
On Mon, Jan 03, 2005 at 10:40:00AM +0100, Jindrich Makovicka wrote:
> Reimar Döffinger wrote:
> >>The demuxer works (at least for me). I also vote for removing
> >>the code which kills the first frame (patch attached).
> >
> >Ok, I removed it. I'll share the cola with you ;-).
> 
> I encountered an .avi file which still fails and sets the bogus 
> samplerate. Maybe it would be better to read first two frames during the 
> initialization, so the (possibly) bogus parameters in the first frame 
> are ignored. The attached patch fixes that issue for me.

Please upload the sample. The samplerate if the first six headers are
checked to match by the demuxer, so if there are bogus parameters in the
first frame the demuxer should have made sure it will be skipped.

Greetings,
Reimar Döffinger