[FFmpeg-devel] [PATCH]Ignore xing number of frames if filesize is?wrong

Michael Niedermayer michaelni at gmx.at
Thu Jul 24 15:00:02 CEST 2014


On Thu, Jul 24, 2014 at 12:50:55PM +0200, Carl Eugen Hoyos wrote:
> On Saturday 19 July 2014 04:40:47 pm Joakim Plate wrote:
> > Carl Eugen Hoyos <cehoyos <at> ag.or.at> writes:
> > > Hi!
> > >
> > > Attached patch fixes ticket #3777 for me, analyzed by Oliver Fromme.
> >
> > What about streaming growing files? It's pretty good to know number of
> > frames of the complete mp3 at start of playback?
> 
> This is true (ticket #2590 contains an example).
> 
> New patch attached.
> 
> Please review, Carl Eugen

>  mp3dec.c |   15 ++++++++++++++-
>  1 file changed, 14 insertions(+), 1 deletion(-)
> e354c8374f6f9655a2a4c1904636218cc4bc9ee6  patchmp3duration2.diff
> diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c
> index 8335388..4016e94 100644

breaks fate

stddev: 6864.49 PSNR: 19.60 MAXDIFF:40809 bytes:   882432/   884736
Test mp3-float-extra_overread failed. Look at tests/data/fate/mp3-float-extra_overread.err for details.
make: *** [fate-mp3-float-extra_overread] Error 1

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140724/5ded2d2f/attachment.asc>


More information about the ffmpeg-devel mailing list