On Mon, Jan 07, 2013 at 01:23:34PM +0100, Anton Khirnov wrote:
> --- a/libavcodec/mimic.c
> +++ b/libavcodec/mimic.c
> @@ -214,7 +215,7 @@ static int vlc_decode_block(MimicContext *ctx, int 
> num_coeffs, int qscale)
>  
>          coeff = vlcdec_lookup[num_bits][value];
> -        if(pos<3)
> +        if (pos<3)

spaces around <

> @@ -293,10 +295,10 @@ static int decode(MimicContext *ctx, int quality, int 
> num_coeffs,
>  static void prepare_avpic(MimicContext *ctx, AVPicture *dst, AVPicture *src)
>  {
>      int i;
> -    dst->data[0] = src->data[0]+( ctx->avctx->height    -1)*src->linesize[0];
> -    dst->data[1] = src->data[2]+((ctx->avctx->height>>1)-1)*src->linesize[2];
> +    dst->data[0] = src->data[0] + ( ctx->avctx->height       - 1) * 
> src->linesize[0];
> +    dst->data[1] = src->data[2] + ((ctx->avctx->height >> 1) - 1) * 
> src->linesize[2];

I'd drop the space after (.  Feel free to ignore.

> @@ -341,23 +343,24 @@ static int mimic_decode_frame(AVCodecContext *avctx, 
> void *data,
>          avctx->width   = width;
>          avctx->height  = height;
>          avctx->pix_fmt = AV_PIX_FMT_YUV420P;
> -        for(i = 0; i < 3; i++) {
> +        for (i = 0; i < 3; i++) {
>              ctx->num_vblocks[i] = -((-height) >> (3 + !!i));
>              ctx->num_hblocks[i] =     width   >> (3 + !!i) ;

stray space before ;

>      ctx->buf_ptrs[ctx->cur_index].reference = 1;
> -    ctx->buf_ptrs[ctx->cur_index].pict_type = is_pframe ? 
> AV_PICTURE_TYPE_P:AV_PICTURE_TYPE_I;
> +    ctx->buf_ptrs[ctx->cur_index].pict_type = is_pframe ? AV_PICTURE_TYPE_P :
> +                                                          AV_PICTURE_TYPE_I;

I'd move the : to the previous line below the ?.  Feel free to ignore.

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

Reply via email to