diff options
Diffstat (limited to 'src/corelib')
-rw-r--r-- | src/corelib/tools/qline.cpp | 11 | ||||
-rw-r--r-- | src/corelib/tools/qline.h | 11 |
2 files changed, 11 insertions, 11 deletions
diff --git a/src/corelib/tools/qline.cpp b/src/corelib/tools/qline.cpp index ebcbba0195..1eb645501c 100644 --- a/src/corelib/tools/qline.cpp +++ b/src/corelib/tools/qline.cpp @@ -517,12 +517,13 @@ QDataStream &operator>>(QDataStream &stream, QLine &line) \fn void QLineF::setLength(qreal length) Sets the length of the line to the given \a length. QLineF will - move the end point - p2() - of the line to give the line its new length. + move the end point - p2() - of the line to give the line its new + length, unless length() was previously zero, in which case no + scaling is attempted. For lines with very short lengths + (represented by denormal floating-point values), results may be + imprecise. - A null line will not be rescaled. For non-null lines with very short lengths - (represented by denormal floating-point values), results may be imprecise. - - \sa length(), isNull(), unitVector() + \sa length(), unitVector() */ /*! diff --git a/src/corelib/tools/qline.h b/src/corelib/tools/qline.h index 4948559509..9b544f2e7e 100644 --- a/src/corelib/tools/qline.h +++ b/src/corelib/tools/qline.h @@ -372,12 +372,11 @@ constexpr inline QPointF QLineF::center() const inline void QLineF::setLength(qreal len) { - if (isNull()) - return; - Q_ASSERT(length() > 0); - const QLineF v = unitVector(); - len /= v.length(); // In case it's not quite exactly 1. - pt2 = QPointF(pt1.x() + len * v.dx(), pt1.y() + len * v.dy()); + const qreal oldLength = length(); + // Scale len by dx() / length() and dy() / length(), two O(1) quantities, + // rather than scaling dx() and dy() by len / length(), which might overflow. + if (oldLength > 0) + pt2 = QPointF(pt1.x() + len * (dx() / oldLength), pt1.y() + len * (dy() / oldLength)); } constexpr inline QPointF QLineF::pointAt(qreal t) const |