[FFmpeg-cvslog] r14612 - trunk/tests/seek.regression.ref
Diego Biurrun
diego
Tue Aug 5 10:27:20 CEST 2008
On Tue, Aug 05, 2008 at 02:46:27AM +0200, Aurelien Jacobs wrote:
>
> Sorry for the mail bombing...
> I wanted to do a major cleanup of matroskadec since a long time and
> I finally got to it.
> I tried hard to keep each change small and self contained, hence the
> huge number of commit.
>
> Anyway, the cleanup was worth it. Just some numbers:
>
> ** before **
> $ wc matroskadec.c
> 3212 8783 107324 matroskadec.c
> $ size matroskadec.o
> text data bss dec hex filename
> 39334 64 0 39398 99e6 matroskadec.o
>
> ** after **
> $ wc matroskadec.c
> 1621 5140 57271 matroskadec.c
> $ size matroskadec.o
> text data bss dec hex filename
> 18748 64 0 18812 497c matroskadec.o
>
> => both object size and lines of code divided by 2...
Not bad at all. How does the libavformat demuxer compare to the MPlayer
demuxer you wrote a long time ago?
BTW, I think (some of) the comments in matroskadec could be doxygenized.
Diego
More information about the ffmpeg-cvslog
mailing list