ArdourCanvas::Line::covers() uses distance_to_segment_squared()
This commit is contained in:
parent
4780c84252
commit
77a63c2bf7
@ -151,18 +151,47 @@ Line::set_y1 (Coord y1)
|
|||||||
bool
|
bool
|
||||||
Line::covers (Duple const & point) const
|
Line::covers (Duple const & point) const
|
||||||
{
|
{
|
||||||
Duple p = canvas_to_item (point);
|
const Duple p = canvas_to_item (point);
|
||||||
|
static const Distance threshold = 2.0;
|
||||||
|
|
||||||
/* compute area of triangle computed by the two line points and the one
|
/* this quick check works for vertical and horizontal lines, which are
|
||||||
we are being asked about. If zero (within a given tolerance), the
|
* common.
|
||||||
points are co-linear and the argument is on the line.
|
|
||||||
*/
|
*/
|
||||||
|
|
||||||
double area = fabs (_points[0].x * (_points[0].y - p.y)) +
|
if (_points[0].x == _points[1].x) {
|
||||||
(_points[1].x * (p.y - _points[0].y)) +
|
/* line is vertical, just check x coordinate */
|
||||||
(p.x * (_points[0].y - _points[1].y));
|
return fabs (_points[0].x - p.x) <= threshold;
|
||||||
|
}
|
||||||
|
|
||||||
if (area < 0.001) {
|
if (_points[0].y == _points[1].y) {
|
||||||
|
/* line is horizontal, just check y coordinate */
|
||||||
|
return fabs (_points[0].y - p.y) <= threshold;
|
||||||
|
}
|
||||||
|
|
||||||
|
Duple at;
|
||||||
|
double t;
|
||||||
|
Duple a (_points[0]);
|
||||||
|
Duple b (_points[1]);
|
||||||
|
const Rect visible (_canvas->visible_area());
|
||||||
|
|
||||||
|
/*
|
||||||
|
Clamp the line endpoints to the visible area of the canvas. If we do
|
||||||
|
not do this, we have a line segment extending to COORD_MAX and our
|
||||||
|
math goes wrong.
|
||||||
|
*/
|
||||||
|
|
||||||
|
a.x = min (a.x, visible.x1);
|
||||||
|
a.y = min (a.y, visible.y1);
|
||||||
|
b.x = min (b.x, visible.x1);
|
||||||
|
b.y = min (b.y, visible.y1);
|
||||||
|
|
||||||
|
double d = distance_to_segment_squared (p, a, b, t, at);
|
||||||
|
|
||||||
|
if (t < 0.0 || t > 1.0) {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (d < threshold) {
|
||||||
return true;
|
return true;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user