[FFmpeg-devel] [PATCH] remove unused and broken test program in libavutil/base64.c

Diego Biurrun diego
Sun Jan 25 10:42:53 CET 2009


On Sat, Jan 24, 2009 at 09:48:16PM +0100, Stefano Sabatini wrote:
> On date Friday 2009-01-23 09:53:41 +0100, Diego Biurrun encoded:
> > On Thu, Jan 22, 2009 at 12:09:19AM +0100, Stefano Sabatini wrote:
> > > On date Thursday 2009-01-22 00:04:13 +0100, Diego Biurrun encoded:
> > > > I found some more cruft in the attic: a forgotten test program in
> > > > libavutil/base64.c.  It never compiled, not even the day it was
> > > > committed, since it calls av_base64_encode() with a wrong number
> > > > of parameters.
> > > > 
> > > > OK to remove?
> > > 
> > > Why not to simply fix it?
> > 
> > The question is whether it is worth the trouble.  Given that it is
> > proven that nobody ever used this program, I believe it to be
> > doubtful that anybody will in the future.
> > 
> > Of course you can have a stab at fixing it...
> 
> It turned out in a semi-complete rewrite of the test program, anyway I
> don't feel like to send a patch and discuss every single change, if
> it's possible to simply replace the new code I'll happily do it.
> 
> Follows the new test code:
> ----------8<--------------------8<--------------------------------
> //#define TEST
> #ifdef TEST

You don't need the #define, just add the file to the TESTS line in
the Makefile.

> int main()

void

Diego




More information about the ffmpeg-devel mailing list