[FFmpeg-cvslog] r14091 - trunk/libavcodec/ra288.c

[Vitor Sessak]

M?ns Rullg?rd wrote:
> vitor <subversion at mplayerhq.hu> writes:
> 
>> Author: vitor
>> Date: Sun Jul  6 14:01:59 2008
>> New Revision: 14091
>>
>> Log:
>> Slighly faster operation
>>
>> Modified:
>>    trunk/libavcodec/ra288.c
>>
>> Modified: trunk/libavcodec/ra288.c
>> ==============================================================================
>> --- trunk/libavcodec/ra288.c	(original)
>> +++ trunk/libavcodec/ra288.c	Sun Jul  6 14:01:59 2008
>> @@ -196,7 +196,7 @@ static void update(Real288_internal *glo
>>
>>      y = glob->phase + 1;
>>      for (x=0; x < 8; x++)
>> -        buffer2[x] = glob->history[(y++) % 8];
>> +        buffer2[x] = glob->history[(y++) & 7];
> 
> If this makes any difference, y is signed.  Is that really correct?

When this code is reached, glob->phase is always 3. So y will always be 
positive and & 7 and %8 should be equivalent...

-Vitor