[FFmpeg-soc] [soc]: r632 - in eac3: ac3.c ac3.h ac3tab.c ac3tab.h eac3.h eac3_parser.c eac3dec.c

Diego Biurrun diego at biurrun.de
Wed Aug 8 20:30:43 CEST 2007


On Wed, Aug 08, 2007 at 04:54:38PM +0200, bwolowiec wrote:
> 
> Log:
> Channel Coupling
> 
> --- eac3/ac3.c	(original)
> +++ eac3/ac3.c	Wed Aug  8 16:54:38 2007
> @@ -489,7 +489,7 @@ static void apply_dithering(int nchans, 
>  
> -void ff_ac3_get_transform_coeffs(GetBitContext *gb, uint8_t (*bap)[AC3_MAX_COEFS], uint8_t (*dexps)[AC3_MAX_COEFS], int nchans, int *chincpl, int *dithflag, float (*transform_coeffs)[AC3_MAX_COEFS], int *strtmant, int *endmant, AVRandomState *dith_state)
> +void ff_ac3_get_transform_coeffs(GetBitContext *gb, uint8_t (*bap)[AC3_MAX_COEFS], uint8_t (*dexps)[AC3_MAX_COEFS], int nchans, int *chincpl, int *dithflag, float (*transform_coeffs)[AC3_MAX_COEFS], int *strtmant, int *endmant, AVRandomState *dith_state, int ncplbnd, int *cplbndstrc, float (*cplco)[18])

These are a lot of parameters...

> --- eac3/eac3_parser.c	(original)
> +++ eac3/eac3_parser.c	Wed Aug  8 16:54:38 2007
> @@ -19,16 +19,18 @@
>  #ifdef DEBUG
> -#define GET_BITS(a, gbc, n) a = get_bits(gbc, n); av_log(NULL, AV_LOG_INFO, "%s: %i\n", __STRING(a), a)
> -#define GET_SBITS(a, gbc, n) a = get_sbits(gbc, n); av_log(NULL, AV_LOG_INFO, "%s: %i\n", __STRING(a), a)
> +#define GET_BITS(a, gbc, n) {a = get_bits(gbc, n); av_log(NULL, AV_LOG_INFO, "%s: %i\n", __STRING(a), a);}
> +#define GET_SBITS(a, gbc, n) {a = get_sbits(gbc, n); av_log(NULL, AV_LOG_INFO, "%s: %i\n", __STRING(a), a);}

This could be vertically aligned.

>  #else
>  #define GET_BITS(a, gbc, n) a = get_bits(gbc, n)
>  #define GET_SBITS(a, gbc, n) a = get_sbits(gbc, n)

same here

> @@ -503,6 +504,7 @@ int ff_eac3_parse_audblk(GetBitContext *
>      }
> +    s->dithflag[CPL_CH] = s->dithflag[s->lfe_channel] = 0;
>      /* These fields for dynamic range control */

These fields "are" for dynamic range control?

> @@ -1133,6 +1155,13 @@ int ff_eac3_parse_audblk(GetBitContext *
>  
>      /* These fields for inclusion of unused dummy data */

Same here, "are for"?

> @@ -1199,7 +1227,7 @@ int ff_eac3_parse_audblk(GetBitContext *
>      /* These fields for quantized mantissa values */

and here

Please don't leave the verbs out of your sentences, it's a sure way to
make them impossible to understand.

Diego



More information about the FFmpeg-soc mailing list