On Mon, Jan 07, 2013 at 01:23:24PM +0100, Anton Khirnov wrote:
> --- a/libavcodec/eatgv.c
> +++ b/libavcodec/eatgv.c
> @@ -72,57 +75,57 @@ static int unpack(const uint8_t *src, const uint8_t 
> *src_end, unsigned char *dst
>              } else {  // 10
> -                size1 = ( ( src[1] & 0xC0) >> 6 );
> +                size1  = ( ( src[1] & 0xC0) >> 6 );

Drop the spaces inside the () while you're at it.

> -        if (size1>0) {
> +        if (size1 > 0) {
>              size -= size1;
> -            run = FFMIN(size1, dst_end-dst);
> +            run   = FFMIN(size1, dst_end-dst);

spaces around -

>              size -= size2;
> -            run = FFMIN(size2, dst_end-dst);
> +            run   = FFMIN(size2, dst_end-dst);

same

> @@ -135,7 +138,9 @@ static int unpack(const uint8_t *src, const uint8_t 
> *src_end, unsigned char *dst
>   * Decode inter-frame
>   * @return 0 on success, -1 on critical buffer underflow
>   */
> -static int tgv_decode_inter(TgvContext * s, const uint8_t *buf, const 
> uint8_t *buf_end){
> +static int tgv_decode_inter(TgvContext * s, const uint8_t *buf,
> +                            const uint8_t *buf_end)
> +{

*s

The rest looks OK.

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

Reply via email to