[FFmpeg-soc] [soc]: r4028 - in wmapro: wma3.h wma3dec.c

Diego Biurrun diego at biurrun.de
Sun Feb 8 21:31:35 CET 2009


On Sun, Feb 08, 2009 at 09:21:26PM +0100, faust3 wrote:
> 
> Log:
> consistently use num as abbreviation for number of

good idea

> --- wmapro/wma3dec.c	Sun Feb  8 20:50:31 2009	(r4027)
> +++ wmapro/wma3dec.c	Sun Feb  8 21:21:25 2009	(r4028)
> @@ -52,7 +52,7 @@ static void dump_context(WMA3DecodeConte
>      PRINT("len prefix",s->len_prefix);
> -    PRINT("nb channels",s->nb_channels);
> +    PRINT("nb channels",s->num_channels);

Why did you leave the nb in place?

> @@ -112,7 +112,7 @@ static av_cold int wma3_decode_end(AVCod
>  
>      if(s->def_decorrelation_mat){
> -        for(i=1;i<=s->nb_channels;i++)
> +        for(i=1;i<=s->num_channels;i++)

These would be more readable with some spaces...

Diego



More information about the FFmpeg-soc mailing list