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

Nehal nehalmistry at gmx.net
Wed Mar 31 05:30:37 CEST 2004


On Tue, 30 Mar 2004 22:11:42 -0500
D Richard Felker III <dalias at aerifal.cx> wrote:

> On Tue, Mar 30, 2004 at 06:10:57PM -0800, Nehal wrote:
> > Hi Albeu,
> > 
> > i downloaded the broken-first-frame.mp3 and tested it,
> > apparently the starting position skips the header if it
> > doesn't start at the beginning of the file. it then searches
> > for the next header data(which in this case, it finds what it
> > thinks is a frame header inside the audio data of first frame
> > which is why it incorrectly detects it as 48000)
> > 
> > anyways here is a revised patch, plays the mp3 fine now
> > 
> > Nehal
> 
> Thanks for debugging the problem! Unfortunately your patch is
> corrupt though (full of ^M's :).
> 
> Rich

it's CF for win32, but i thought 'patch' strips these
automatically, hmm, oh well, anyways here it is, again, hopefully
it's ok

Nehal

-------------- next part --------------
A non-text attachment was scrubbed...
Name: mp3skip-revision2.diff
Type: application/octet-stream
Size: 1640 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040330/060f81a8/attachment.obj>


More information about the MPlayer-dev-eng mailing list