On 22/02/14 21:34, Guillaume Martres wrote:
> On Wednesday 19 February 2014 22:43:13 Luca Barbato wrote:
>> +                    if (x < 0)
>> +                        x_pos = -((-x / 2) << s->sps->pixel_shift);
> 
> This seems very weird to me, x and y are used to get an index inside the 
> chroma frame so x should never be negative.

x can be negative and gets as negative as -8.

I redid the patch in a nicer way (and should be OMGfaster for those
counting cycles).

lu

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to