[FFmpeg-soc] [soc]: r1511 - amr/amrnbfloatdec.c

Diego Biurrun diego at biurrun.de
Tue Dec 4 01:38:58 CET 2007


On Tue, Dec 04, 2007 at 01:35:28AM +0100, superdump wrote:
> 
> --- amr/amrnbfloatdec.c	(original)
> +++ amr/amrnbfloatdec.c	Tue Dec  4 01:35:28 2007
> @@ -722,16 +722,16 @@ static void decode_8_pulses_31bits(int16
>      // coded using 7+3 bits with the 3 LSBs being, individually, the LSB of 1 of
>      // the 3 pulses and the upper 7 bits being coded in base 5
>      temp = fixed_index[4] >> 3;
> -    pulse_position[0] = (temp    %5)<<1 +  fixed_index[4]    &1;
> -    pulse_position[4] = (temp /5)%5)<<1 + (fixed_index[4]>>1)&1;
> -    pulse_position[1] = (temp/25)%5)<<1 + (fixed_index[4]>>2)&1;
> +    pulse_position[0] = ((temp    %5)<<1) + ( fixed_index[4]    &1);
> +    pulse_position[4] = ((temp /5)%5)<<1) + ((fixed_index[4]>>1)&1);
> +    pulse_position[1] = ((temp/25)%5)<<1) + ((fixed_index[4]>>2)&1);

Here (and everywhere else) I think some spaces around operators would
improve readability.

Diego



More information about the FFmpeg-soc mailing list