From 1ef73a70b87edb6eb6b065014b1a68c8fde2d2e2 Mon Sep 17 00:00:00 2001
From: Craig Watson <craig@watsons.ch>
Date: Tue, 5 Apr 2016 15:27:31 +0200
Subject: [PATCH] Strokes: corrected tracking of last received point

---
 src/domain/UBGraphicsStroke.cpp | 7 ++++---
 src/domain/UBGraphicsStroke.h   | 2 ++
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/domain/UBGraphicsStroke.cpp b/src/domain/UBGraphicsStroke.cpp
index 90db6dde..3553a465 100644
--- a/src/domain/UBGraphicsStroke.cpp
+++ b/src/domain/UBGraphicsStroke.cpp
@@ -71,6 +71,7 @@ QList<QPointF> UBGraphicsStroke::addPoint(const QPointF& point, UBInterpolator::
     int n = mDrawnPoints.size();
 
     if (interpolationMethod == UBInterpolator::NoInterpolation || n == 0) {
+        mLastReceivedPoint = point;
         mDrawnPoints << point;
         mAllPoints << point;
         return QList<QPointF>() << point;
@@ -80,8 +81,7 @@ QList<QPointF> UBGraphicsStroke::addPoint(const QPointF& point, UBInterpolator::
         // This is a bit special, as the curve we are interpolating is not between two drawn points;
         // it is between the midway points of the second-to-last and last point, and last and current point.
         qreal MIN_DISTANCE = 3; // TODO: make this dependant on zoom
-        QPointF lastPoint = mDrawnPoints.last();
-        qreal distance = QLineF(lastPoint, point).length();
+        qreal distance = QLineF(mLastReceivedPoint, point).length();
 
         //qDebug() << "distance: " << distance;
 
@@ -89,7 +89,7 @@ QList<QPointF> UBGraphicsStroke::addPoint(const QPointF& point, UBInterpolator::
         // look smoother (e.g shaking slightly won't affect the curve).
         if (distance < MIN_DISTANCE) {
             // we still keep track of that point to calculate the distance correctly next time around
-            mDrawnPoints << point;
+            mLastReceivedPoint = point;
             return QList<QPointF>();
         }
 
@@ -101,6 +101,7 @@ QList<QPointF> UBGraphicsStroke::addPoint(const QPointF& point, UBInterpolator::
             return QList<QPointF>() << ((lastPoint + point)/2.0);
         }
 
+
         QPointF p0 = mDrawnPoints[mDrawnPoints.size() - 2];
         QPointF p1 = mDrawnPoints[mDrawnPoints.size() - 1];
         QPointF p2 = point;
diff --git a/src/domain/UBGraphicsStroke.h b/src/domain/UBGraphicsStroke.h
index 2f9f4eae..5ef91507 100644
--- a/src/domain/UBGraphicsStroke.h
+++ b/src/domain/UBGraphicsStroke.h
@@ -76,6 +76,8 @@ class UBGraphicsStroke
         /// All the points (including interpolated) that are used to draw the stroke
         QList<QPointF> mAllPoints;
 
+        QPointF mLastReceivedPoint;
+
 };
 
 #endif /* UBGRAPHICSSTROKE_H_ */