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

Ramiro Polla ramiro.polla
Sat Aug 29 17:30:17 CEST 2009


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 @@
>      bi->bmiHeader.biWidth  = width ;
>      bi->bmiHeader.biHeight = height;
>
> -#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
> -
>      ret = SendMessage(ctx->hwnd, WM_CAP_SET_VIDEOFORMAT, bisize, (LPARAM) bi);
>      if(!ret) {
>          av_log(s, AV_LOG_ERROR, "Could not set Video Format.\n");

I prefer keeping this around.

Ramiro Polla



More information about the ffmpeg-devel mailing list