[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