[Ffmpeg-devel] [PATCH] BMP encoder

[Diego Biurrun]

On Mon, Oct 30, 2006 at 04:05:39PM +0100, Michel Bardiaux wrote:
> Currently, only straight RGB.
> 
> --- libavcodec/bmp.c	(revision 6838)
> +++ libavcodec/bmp.c	(working copy)
> @@ -32,10 +34,15 @@
>  
> +// Hence the uncondfitional byteswap.

typo

> --- libavcodec/amr.c	(revision 6838)
> +++ libavcodec/amr.c	(working copy)
> @@ -57,6 +57,8 @@
>  
>  #include "avcodec.h"
>  
> +#if defined(CONFIG_AMR_NB) || defined(CONFIG_AMR_NB_FIXED)
> +
>  #ifdef CONFIG_AMR_NB_FIXED
>  
>  #define MMS_IO
> @@ -494,8 +496,6 @@
>  
>  #endif
>  
> -#if defined(CONFIG_AMR_NB) || defined(CONFIG_AMR_NB_FIXED)
> -
>  AVCodec amr_nb_decoder =
>  {
>      "amr_nb",

unrelated hunk

> --- libavcodec/allcodecs.c	(revision 6838)
> +++ libavcodec/allcodecs.c	(working copy)
> @@ -134,6 +134,7 @@
>  #ifdef CONFIG_PNG_ENCODER
>      register_avcodec(&png_encoder);
>  #endif //CONFIG_PNG_ENCODER
> +    register_avcodec(&bmp_encoder);
>  #endif
>  #ifdef CONFIG_PPM_ENCODER
>      register_avcodec(&ppm_encoder);

This should be surrounded by proper #ifdef.  Also, put this next to the
BMP decoder.

Diego