diff options
| author | Edward Welbourne <edward.welbourne@qt.io> | 2017-02-23 16:49:16 +0100 |
|---|---|---|
| committer | Edward Welbourne <edward.welbourne@qt.io> | 2017-02-24 11:56:56 +0000 |
| commit | 9310c07aac07efcdeafe83ada1e235b0b99e19e4 (patch) | |
| tree | c3d418b9cc8712b4723bb66708d797b19e84c3b3 /src | |
| parent | 9012942665257b514c7356ea52c7612bba5a4ef9 (diff) | |
Use qDegreesToRadians() more widely
It documents intent more clearly (and saves us a hand-rolled value for pi).
Task-number: QTBUG-58083
Change-Id: I34871ad8526087aa892afdcc85b362bc7442bad1
Reviewed-by: Miikka Heikkinen <miikka.heikkinen@qt.io>
Diffstat (limited to 'src')
| -rw-r--r-- | src/charts/domain/polardomain.cpp | 4 | ||||
| -rw-r--r-- | src/charts/piechart/piesliceitem.cpp | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/charts/domain/polardomain.cpp b/src/charts/domain/polardomain.cpp index 1d5154e4..3c6e6b57 100644 --- a/src/charts/domain/polardomain.cpp +++ b/src/charts/domain/polardomain.cpp @@ -89,8 +89,8 @@ QVector<QPointF> PolarDomain::calculateGeometryPoints(const QVector<QPointF> &ve QPointF PolarDomain::polarCoordinateToPoint(qreal angularCoordinate, qreal radialCoordinate) const { - qreal dx = qSin(angularCoordinate * (M_PI / 180)) * radialCoordinate; - qreal dy = qCos(angularCoordinate * (M_PI / 180)) * radialCoordinate; + qreal dx = qSin(qDegreesToRadians(angularCoordinate)) * radialCoordinate; + qreal dy = qCos(qDegreesToRadians(angularCoordinate)) * radialCoordinate; return QPointF(dx, -dy); } diff --git a/src/charts/piechart/piesliceitem.cpp b/src/charts/piechart/piesliceitem.cpp index 19d6e4ef..05a790e4 100644 --- a/src/charts/piechart/piesliceitem.cpp +++ b/src/charts/piechart/piesliceitem.cpp @@ -43,8 +43,8 @@ QT_CHARTS_BEGIN_NAMESPACE QPointF offset(qreal angle, qreal length) { - qreal dx = qSin(angle * (M_PI / 180)) * length; - qreal dy = qCos(angle * (M_PI / 180)) * length; + qreal dx = qSin(qDegreesToRadians(angle)) * length; + qreal dy = qCos(qDegreesToRadians(angle)) * length; return QPointF(dx, -dy); } |