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

Robert Swain robert.swain
Sat Sep 13 22:26:21 CEST 2008


2008/9/13 Uoti Urpala <uoti.urpala at pp1.inet.fi>:
> On Sat, 2008-09-13 at 15:54 -0300, Ramiro Polla wrote:
>> > -    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);
>
>> Is there anything in the C standard that specifies the order these
>> functions must be called when they're on the same line?
>
> There isn't. "Same line" is not significant, but the arguments of the
> "+" operator could be evaluated in any order.
>
>> Or is it possible that a compiler might screw up here?
>
> The code is screwed up, so it's not the compiler's fault if the result
> doesn't work.

Fixed.

Rob




More information about the ffmpeg-cvslog mailing list