[PATCH] randr: Fix logic in RRPointerToNearestCrtc

walter harms wharms at bfs.de
Thu Jul 31 00:20:22 PDT 2014



Am 30.07.2014 23:38, schrieb Keith Packard:
> walter harms <wharms at bfs.de> writes:
> 
>> distance is normally defined with sqrt(dx*dx+dy*dy)
>> perhaps you can use dx*dx instead of abs() to avoid
>> confusion about dist ?
> 
> That introduces all kinds of range adventures; not sure it's worth the
> trouble?
> 

for now there is no check for any integer overflow either, so i assume that
the values are sane.
Having a common pattern make it more easy to spot trouble for human beings.
NTL the maintainer has to decide. I can only comment.

re,
 wh


More information about the xorg-devel mailing list