On Thu, Jan 19, 2012 at 07:40:12PM +0530, Aneesh Dogra wrote:
>
> --- a/libavcodec/gif.c
> +++ b/libavcodec/gif.c
> @@ -71,10 +71,10 @@ static int gif_image_write_header(AVCodecContext *avctx,
>
> /* the global palette */
> - for(i=0;i<256;i++) {
> + for(i = 0; i < 256; i++) {
for (
> gif_image_write_header(avctx, &outbuf_ptr, (uint32_t *)pict->data[1]);
> - gif_image_write_image(avctx, &outbuf_ptr, end, pict->data[0],
> pict->linesize[0]);
> + gif_image_write_image (avctx, &outbuf_ptr, end, pict->data[0],
> + pict->linesize[0]);
No spaces before function name and '(', not even to make things line up.
> @@ -174,6 +179,6 @@ AVCodec ff_gif_encoder = {
> .encode = gif_encode_frame,
> .close = gif_encode_close,
> - .pix_fmts= (const enum PixelFormat[]){PIX_FMT_RGB8, PIX_FMT_BGR8,
> PIX_FMT_RGB4_BYTE, PIX_FMT_BGR4_BYTE, PIX_FMT_GRAY8, PIX_FMT_PAL8,
> PIX_FMT_NONE},
> - .long_name= NULL_IF_CONFIG_SMALL("GIF (Graphics Interchange Format)"),
> + .pix_fmts = (const enum PixelFormat[]){PIX_FMT_RGB8, PIX_FMT_BGR8,
> PIX_FMT_RGB4_BYTE, PIX_FMT_BGR4_BYTE, PIX_FMT_GRAY8, PIX_FMT_PAL8,
> PIX_FMT_NONE},
> + .long_name = NULL_IF_CONFIG_SMALL("GIF (Graphics Interchange
> Format)"),
You could break those lines while you're at it.
Diego
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
