On Tue, Oct 26, 2010 at 1:07 AM, Keith Packard <[email protected]> wrote:
> On Mon, 25 Oct 2010 11:51:46 +0530, Siddhesh Poyarekar 
> <[email protected]> wrote:
>
>> -    if (slopes[j].dy * slopes[i].dx > slopes[i].dy * slopes[j].dx)
>> +    if ((long long)slopes[j].dy * slopes[i].dx > (long long)slopes[i].dy * 
>> slopes[j].dx)
>
> Looks like the fix is in the right place, but I'd prefer int64_t instead of 
> 'long long' here.
>

Thanks for the review. I have sent a modified patch.


-- 
Siddhesh Poyarekar
http://siddhesh.in
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to