[FFmpeg-cvslog] r15311 - trunk/libavcodec/aac.c

[Ramiro Polla]

superdump wrote:
> Author: superdump
> Date: Sat Sep 13 20:47:43 2008
> New Revision: 15311
> 
> Log:
> Correct order of parsing for pulse scalefactor band and offset to match the
> specification.
> 
> Patch by Alex Converse (alex converse gmail com)
> 
> 
> Modified:
>    trunk/libavcodec/aac.c
> 
> Modified: trunk/libavcodec/aac.c
> ==============================================================================
> --- trunk/libavcodec/aac.c	(original)
> +++ trunk/libavcodec/aac.c	Sat Sep 13 20:47:43 2008
> @@ -597,7 +597,7 @@ static int decode_scalefactors(AACContex
>  static void decode_pulses(Pulse * pulse, GetBitContext * gb, const uint16_t * swb_offset) {
>      int i;
>      pulse->num_pulse = get_bits(gb, 2) + 1;
> -    pulse->pos[0]    = get_bits(gb, 5) + swb_offset[get_bits(gb, 6)];
> +    pulse->pos[0]    = swb_offset[get_bits(gb, 6)] + get_bits(gb, 5);
>      pulse->amp[0]    = get_bits(gb, 4);
>      for (i = 1; i < pulse->num_pulse; i++) {
>          pulse->pos[i] = get_bits(gb, 5) + pulse->pos[i-1];

Is there anything in the C standard that specifies the order these 
functions must be called when they're on the same line?

Or is it possible that a compiler might screw up here?