Bug 342642

Git commit 16d4aea302e9f9eeca86e3ad72f448b8303d31b5 by Barth Netterfield.
Committed on 11/01/2015 at 05:19.
Pushed by netterfield into branch 'master'.

Fix some qreal issues.

floating point values which count pixels or screen locations
can and should be qreal.

floating point values which are
involved with computational manipulation of the data
should be double, even though this is slower in ARM.

They shouldn't mix if it can be helped.

M  +1    -1    src/libkst/editablevector.cpp
M  +4    -4    src/libkst/vector.cpp
M  +1    -1    src/libkstmath/curve.cpp
M  +1    -1    src/plugins/dataobject/activitylevel/activitylevel.cpp

http://commits.kde.org/kst-plot/16d4aea302e9f9eeca86e3ad72f448b8303d31b5

Thanks, but I still get errors with:
/builddir/build/BUILD/Kst-2.0.8/src/libkst/vector.cpp:598:42: error: no matching function for call to 'qMax(double&, qreal)'
           _minPos=qMax(_minPos,qreal(0.0));
/builddir/build/BUILD/Kst-2.0.8/src/libkst/vector.cpp:602:45: error: no matching function for call to 'qMax(double&, qreal)'
           _minPos=qMin(qMax(_v[i],qreal(0.0)),_minPos);

And after those:
/builddir/build/BUILD/Kst-2.0.8/src/libkstmath/curve.cpp:687:83: error: no matching function for call to 'qMax(qreal, double)'
   return qMax(qreal(1.01), PointSize * ((w.width() + w.height()) * (1.0 / 4000.0)));

Both of those were changed in git (origin/master (not 2.0.8, which has already been released)).

Let me now if the changes work there.

Hmm, I seemed to have missed the vector changes somehow, but I think you are missing:

diff --git a/src/libkstmath/curve.cpp b/src/libkstmath/curve.cpp
index 7e8c4bb..79182e6 100644
--- a/src/libkstmath/curve.cpp
+++ b/src/libkstmath/curve.cpp
@@ -684,7 +684,7 @@ RelationPtr Curve::makeDuplicate() const {
 // and not just pulled out of the air by what looks ~good.
 // They are currently "about right" for printing to US Letter.
 double Curve::pointDim(QRectF w) const {
-  return qMax(qreal(1.01), PointSize * ((w.width() + w.height()) * (1.0 / 4000.0)));
+  return qMax(double(1.01), PointSize * ((w.width() + w.height()) * (1.0 / 4000.0)));
 }

 double Curve::lineDim(const QRectF &R, double linewidth) {

Git commit 1365ccf2fd2da59025be623b97b7207f95ba8884 by Barth Netterfield.
Committed on 29/01/2015 at 16:18.
Pushed by netterfield into branch 'master'.

Fix another qreal/double issue

M  +1    -1    src/libkstmath/curve.cpp

http://commits.kde.org/kst-plot/1365ccf2fd2da59025be623b97b7207f95ba8884

Can anyone confirm whether this is still breaking?

Looks good now,
http://koji.fedoraproject.org/koji/buildinfo?buildID=741522