e5ab6a6588
We want to avoid caling toMSecsSinceEpoch() since it's expensive for LocalTime (which is presumed to be the common case). We can do so when both sides have the same offset from UTC (and this can cheaply be determined) but that's no help for two local times months apart, one in DST the other not. However, in this case, the difference in millis is big enough that no plausible difference in offset can overcome it, so we can again avoid toMSecsSinceEpoch() and simply compare millis. This should make some previously-expensive comparisons cheap. Add test-cases to the QDateTime ordering test that verify this doesn't lead to mis-comparison at the biggest offset-difference known. Fixes: QTBUG-131491 Change-Id: I1afd5d058c8663c908f898d4c50d0837549b87db Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de> (cherry picked from commit ef540d77751e24fe0b345694f43cdafca3434c68) Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> |
||
|---|---|---|
| .. | ||
| .gitignore | ||
| CMakeLists.txt | ||
| tst_qdatetime.cpp | ||
| tst_qdatetime_mac.mm | ||