[FFmpeg-cvslog] r14612 - trunk/tests/seek.regression.ref

[Aurelien Jacobs]

Diego Biurrun wrote:

> 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?

Lavf's one is now definitely smaller :-)
It is also more functional. So I plan to make it the default one
in MPlayer.
But before doing this, I have a few more improvements planed.

> BTW, I think (some of) the comments in matroskadec could be doxygenized.

This is on my TODO list, but pretty low.

Aurel