[FFmpeg-devel] [PATCH] remove disabled code part 1

Diego Biurrun diego
Tue Sep 1 14:33:14 CEST 2009


On Sat, Aug 29, 2009 at 12:30:17PM -0300, Ramiro Polla wrote:
> On Thu, Aug 27, 2009 at 7:25 AM, Diego Biurrun<diego at biurrun.de> wrote:
> > On Fri, Aug 07, 2009 at 11:15:19AM +0200, Diego Biurrun wrote:
> >> I started looking into disabled code, there is lots of it and probably
> >> mostly cruft. ?Here is a patch that removes it, starting at the top
> >> level and the tests subdirectory. ?More patches shall follow later.
> >>
> >> I will commit approved hunks only.
> >
> > libavdevice..
> 
> > --- libavdevice/vfwcap.c	(revision 19725)
> > +++ libavdevice/vfwcap.c	(working copy)
> > @@ -294,18 +294,6 @@
> >
> > -#if 0
> > -    /* For testing yet unsupported compressions
> > -     * Copy these values from user-supplied verbose information */
> > -    bi->bmiHeader.biWidth       = 320;
> > -    bi->bmiHeader.biHeight      = 240;
> > -    bi->bmiHeader.biPlanes      = 1;
> > -    bi->bmiHeader.biBitCount    = 12;
> > -    bi->bmiHeader.biCompression = MKTAG('I','4','2','0');
> > -    bi->bmiHeader.biSizeImage   = 115200;
> > -    dump_bih(s, &bi->bmiHeader);
> > -#endif
> 
> I prefer keeping this around.

Maybe #ifdef DEBUG then?

Diego



More information about the ffmpeg-devel mailing list