[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