On Mon, Sep 01, 2014 at 03:18:58PM -0400, Gabriel Dume wrote:
> --- a/libavcodec/asv.c
> +++ b/libavcodec/asv.c
> @@ -30,58 +30,59 @@
>
> const uint8_t ff_asv_ccp_tab[17][2] = {
> - {0xF,5}, //EOB
> + { 0xF, 5 }, //EOB
space after //
> --- a/libavcodec/asvdec.c
> +++ b/libavcodec/asvdec.c
> @@ -35,7 +35,7 @@
>
> -#define VLC_BITS 6
> +#define VLC_BITS 6
> #define ASV2_LEVEL_VLC_BITS 10
Such definitions look slightly more readable if you right-align numbers.
> @@ -69,7 +69,7 @@ static av_cold void init_vlcs(ASV1Context *a)
> -//FIXME write a reversed bitstream reader to avoid the double reverse
> +// FIXME write a reversed bitstream reader to avoid the double reverse
> static inline int asv2_get_bits(GetBitContext *gb, int n)
> {
> return ff_reverse[get_bits(gb, n) << (8-n)];
spaces around -
> --- a/libavcodec/asvenc.c
> +++ b/libavcodec/asvenc.c
> @@ -32,105 +32,138 @@
>
> - put_bits(pb, n, ff_reverse[ v << (8-n) ]);
> + put_bits(pb, n, ff_reverse[v << (8-n)]);
ditto
> - if( (block[index + 0] = (block[index + 0]*a->q_intra_matrix[index +
> 0] + (1<<15))>>16) ) ccp |= 8;
> - if( (block[index + 8] = (block[index + 8]*a->q_intra_matrix[index +
> 8] + (1<<15))>>16) ) ccp |= 4;
> - if( (block[index + 1] = (block[index + 1]*a->q_intra_matrix[index +
> 1] + (1<<15))>>16) ) ccp |= 2;
> - if( (block[index + 9] = (block[index + 9]*a->q_intra_matrix[index +
> 9] + (1<<15))>>16) ) ccp |= 1;
> -
> +
> + if((block[index + 0] = (block[index + 0] *
> + a->q_intra_matrix[index + 0] + (1 << 15)) >>
> 16))
> + ccp |= 8;
> + if((block[index + 8] = (block[index + 8] *
> + a->q_intra_matrix[index + 8] + (1 << 15)) >>
> 16))
> + ccp |= 4;
> + if((block[index + 1] = (block[index + 1] *
> + a->q_intra_matrix[index + 1] + (1 << 15)) >>
> 16))
> + ccp |= 2;
> + if((block[index + 9] = (block[index + 9] *
> + a->q_intra_matrix[index + 9] + (1 << 15)) >>
> 16))
> + ccp |= 1;
if (
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
