According to https://doc.qt.io/qt-5/qtglobal.html#qreal-typedef :
qreal is a "Typedef for double unless Qt is configured with the -qreal float option."
So i am not that happy with configuring qreal as
<define name="qreal" value="double"/>
inside qt.cfg.
Currently it is not configured at all.
But since qreal seems to be heavily used i want to give users the opportunity to configure it.
Maybe via two different library files "qt_qreal_double.cfg" and "qt_qreal_float.cfg" only configuring qreal accordingly that then can then be used?
Or is it OK to use double for qreal inside qt.cfg since this is the default?
Any ideas/opinions on that?
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
According to https://doc.qt.io/qt-5/qtglobal.html#qreal-typedef :
qreal is a "Typedef for double unless Qt is configured with the -qreal float option."
So i am not that happy with configuring qreal as
inside qt.cfg.
Currently it is not configured at all.
But since qreal seems to be heavily used i want to give users the opportunity to configure it.
Maybe via two different library files "qt_qreal_double.cfg" and "qt_qreal_float.cfg" only configuring qreal accordingly that then can then be used?
Or is it OK to use double for qreal inside qt.cfg since this is the default?
Any ideas/opinions on that?
qreal
is now defined asdouble
: https://github.com/danmar/cppcheck/commit/cf44964f206b209f19c11559efb600afb7c0c7afIMHO this is the best solution for now.