[FFmpeg-soc] [soc]: r589 - dirac/libavcodec/dirac.c

[Marco Gerards]

Michael Niedermayer <michaelni at gmx.at> writes:

Hi,

> On Thu, Aug 02, 2007 at 02:17:39PM +0200, marco wrote:
>> Author: marco
>> Date: Thu Aug  2 14:17:38 2007
>> New Revision: 589
>> 
>> Log:
>> the first step towards motion compensation. please not that a lot is missing, it's a hack and this will not work yet
>
> [...]
>> +    xpos = FFMAX(0, FFMIN(x, width  * 2));
>> +    ypos = FFMAX(0, FFMIN(y, height * 2));
>
> av_clip()

Changed.  I also read the other comments you just sent in and made
changes accordingly and committed.


> [...]
>>  
>> -        retire_num = dirac_get_se_golomb(gb) +  s->picnum;
>> +        retire_num = dirac_get_se_golomb(gb) + s->picnum;
>
> could you try to seperate cosmetic from non cosmetic commits in the future
> please

Yes.

--
Marco