Removing accidental use of double instead of qreal from QLine
authormread <qt-info@nokia.com>
Thu, 6 Oct 2011 14:07:29 +0000 (15:07 +0100)
committermread <qt-info@nokia.com>
Thu, 6 Oct 2011 15:38:37 +0000 (16:38 +0100)
commit51ed77637a6001f66805e8091ce8616d8f2530db
treef003f4be14bc08c96d57424511812a27cb6a45da
parentbf69c7e0af67ec877da4fee244386e538fa1c01d
Removing accidental use of double instead of qreal from QLine

QLine contained a number of places where conversions and calculations
were done as double where qreal should have been used instead. This
is caused by the use of raw floating point literals. The fix is to
construct the literals as qreal. The angle conversions are simplified
to reduce the risk of a growing loss of precision from this change.

On Symbian, where qreal is float, this gives a 5-10% performance
improvement to the affected functions, as tested by a new benchmark
test. The auto tests pass, these contain precision tests which indicates
that there is no significant loss of precision with this change.

On Windows, where qreal is double, this has no significant effect on
performance.

Task-number: QTBUG-4894
Reviewed-by: Shane Kearns
src/corelib/tools/qline.cpp
tests/benchmarks/corelib/tools/qline/main.cpp [new file with mode: 0644]
tests/benchmarks/corelib/tools/qline/qline.pro [new file with mode: 0644]
tests/benchmarks/corelib/tools/tools.pro