Re: [Inkscape-devel] patch comments

bulia byak wrote:

            && ( abs( (gint) event->motion.x - event_context->xp ) 

< event_context->tolerance ) && ( abs( (gint) event->motion.y - event_context->yp ) < event_context->tolerance ) ) {

when you see this pattern you should realise that it is a distance comparison. in this case you could replace it with NR::LInfty(d) < event_context->tolerance

I also think it would be worth moving the conversion to NR::Point back into GdkEvent or similar. (not sure how, perhaps inherit from gdkevent and provide an accessor function?)

p.s. you needed to set CVS_RSH=ssh .

njh