> From: Aneesh Dogra <[email protected]>
> To: [email protected]
> Cc: Aneesh Dogra <[email protected]>
> Sent: Saturday, December 31, 2011 6:29 PM
> Subject: [libav-devel] [PATCH 4/8] bfi: K&R cleanup
> 
> ---
> libavcodec/bfi.c |   15 ++++++++-------
> 1 files changed, 8 insertions(+), 7 deletions(-)
> 
> diff --git a/libavcodec/bfi.c b/libavcodec/bfi.c
> index 3c66226..b17c3de 100644
> --- a/libavcodec/bfi.c
> +++ b/libavcodec/bfi.c
> @@ -71,7 +71,7 @@ static int bfi_decode_frame(AVCodecContext * avctx, void 
> *data,

The format we follow is "type *var_name".

>          bfi->frame.pict_type = AV_PICTURE_TYPE_I;
>          bfi->frame.key_frame = 1;
>          /* Setting the palette */
> -        if(avctx->extradata_size>768) {
> +        if (avctx->extradata_size > 768) {
>              av_log(NULL, AV_LOG_ERROR, "Palette is too 
> large.\n");
>              return -1;
>          }
> @@ -91,16 +91,17 @@ static int bfi_decode_frame(AVCodecContext * avctx, void 

Above comment applies here too.

[...]

The rest looks OK to me.

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to