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

[Reimar Döffinger]

Hi,
> > > > > See libmpcodecs/ad_mp3lib.c, line 49. I think there were/are some broken
> > > > > MP3 encoders that create only trash for the first frame or something
> > > > > like that...
> > > > 
> > > > no, the idiotic demuxer was starting at the wrong place in the file
> > > > making the decoder see the wrong data as the "first frame"... there's
> > > > no other decoder that has to discard the first frame when playing
> > > > mp3s, so mplayer shouldn't have to do it either...
> > > 
> > > So the questions is: Does the demuxer now work correctly? Should this be
> > > removed?
> > 
> > afaik, yes, the patch was committed. i'm not positive though..
> 
> 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 ;-).

Greetings,
Reimar Döffinger