[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