X-Git-Url: http://shamusworld.gotdns.org/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fline.cpp;h=880a79d60bc48254645794628ce1cb69dbe246fb;hb=baf67656b97e3d61e9223e66ebe4f554e364cd4a;hp=06fbbc6a28cfe7cffecc8169e0069a58c369828c;hpb=bd9b40058a376c946318a444dd6c77737ec6ac98;p=architektonas

diff --git a/src/line.cpp b/src/line.cpp
index 06fbbc6..880a79d 100644
--- a/src/line.cpp
+++ b/src/line.cpp
@@ -4,7 +4,7 @@
 // (C) 2011 Underground Software
 // See the README and GPLv3 files for licensing and warranty information
 //
-// JLH = James L. Hammons <jlhamm@acm.org>
+// JLH = James Hammons <jlhamm@acm.org>
 //
 // WHO  WHEN        WHAT
 // ---  ----------  ------------------------------------------------------------
@@ -49,10 +49,10 @@ Line::~Line()
 	painter->SetPen(QPen(Qt::red, 2.0, Qt::DotLine));
 
 	if ((state == OSSelected) || ((state == OSInactive) && hitPoint1))
-		painter->DrawEllipse(position, 4.0, 4.0);
+		painter->DrawHandle(position);
 
 	if ((state == OSSelected) || ((state == OSInactive) && hitPoint2))
-		painter->DrawEllipse(endpoint, 4.0, 4.0);
+		painter->DrawHandle(endpoint);
 
 	if ((state == OSInactive) && !hitLine)
 		painter->SetPen(QPen(Qt::black, 1.0, Qt::SolidLine));
@@ -65,16 +65,21 @@ Line::~Line()
 		Vector current(point2 - point1);
 		Vector v = current.Unit() * length;
 		Vector v2 = point1 + v;
-		painter->DrawLine((int)point1.x, (int)point1.y, (int)v2.x, (int)v2.y);
+//		painter->DrawLine((int)point1.x, (int)point1.y, (int)v2.x, (int)v2.y);
+		painter->DrawLine(point1, v2);
 
 		if (current.Magnitude() > length)
 		{
 			painter->SetPen(QPen(QColor(128, 0, 0), 1.0, Qt::DashLine));
-			painter->DrawLine((int)v2.x, (int)v2.y, (int)point2.x, (int)point2.y);
+//			painter->DrawLine((int)v2.x, (int)v2.y, (int)point2.x, (int)point2.y);
+			painter->DrawLine(v2, point2);
 		}
 	}
+// Problem: when drawing at large zoom levels, this throws away precision thus
+//          causing the line to rendered too short. !!! FIX !!! [DONE]
 	else
-		painter->DrawLine((int)position.x, (int)position.y, (int)endpoint.x, (int)endpoint.y);
+//		painter->DrawLine((int)position.x, (int)position.y, (int)endpoint.x, (int)endpoint.y);
+		painter->DrawLine(position, endpoint);
 }
 
 /*virtual*/ Vector Line::Center(void)
@@ -86,8 +91,7 @@ Line::~Line()
 
 /*virtual*/ bool Line::Collided(Vector point)
 {
-// Can't assume this!
-// Actually, we can, since this is a mouse down event here.
+	// We can assume this, since this is a mouse down event here.
 	objectWasDragged = false;
 	HitTest(point);
 
@@ -128,7 +132,7 @@ level container, it won't get drawn...
 But we can fix that by making this object call any attached object's (like
 a dimension only) Draw() function... :-/
 */
-				attachedDimension = new Dimension(&position, &endpoint, this);
+				attachedDimension = new Dimension(&position, &endpoint, DTLinear, this);
 
 				if (parent != NULL)
 					parent->Add(attachedDimension);
@@ -356,6 +360,18 @@ about keeping track of old states...
 		state = oldState;
 }
 
+// Check to see if the point passed in coincides with any we have. If so, return a
+// pointer to it; otherwise, return NULL.
+/*virtual*/ Vector * Line::GetPointAt(Vector v)
+{
+	if (v == position)
+		return &position;
+	else if (v == endpoint)
+		return &endpoint;
+
+	return 0;
+}
+
 #if 0
 void Line::SetDimensionOnPoint1(Dimension * dimension)
 {
@@ -378,7 +394,7 @@ void Line::SetDimensionOnLine(Dimension * dimension/*=NULL*/)
 	// If they don't pass one in, create it for the caller.
 	if (dimension == NULL)
 	{
-		dimension = new Dimension(&position, &endpoint, this);
+		dimension = new Dimension(&position, &endpoint, DTLinear, this);
 
 		if (parent)
 			parent->Add(dimension);
@@ -437,12 +453,13 @@ bool Line::HitTest(Point point)
 	// simplifies the calculation of the determinant.
 
 //How do we determine distance here? Especially if zoomed in or out???
-#warning "!!! Distances tested for may not be valid if zoomed in or out !!!"
-	if (v1.Magnitude() < 8.0)
+//#warning "!!! Distances tested for may not be valid if zoomed in or out !!!"
+// [FIXED]
+	if ((v1.Magnitude() * Painter::zoom) < 8.0)
 		hitPoint1 = true;
-	else if (v2.Magnitude() < 8.0)
+	else if ((v2.Magnitude() * Painter::zoom) < 8.0)
 		hitPoint2 = true;
-	else if (distance < 5.0)
+	else if ((distance * Painter::zoom) < 5.0)
 		hitLine = true;
 
 	return StateChanged();
@@ -503,7 +520,7 @@ rearranging we get:
 t(d1x) - s(d2x) = p2x - p0x
 t(d1y) - s(d2y) = p2y - p0y
 
-Determinant D is ad - bc where the matrix look like:
+Determinant D is ad - bc where the matrix looks like:
 
 a b
 c d