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 _______________________________________________ FFmpeg-soc mailing list [email protected] http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc
