diff options
author | Marc Mutz <marc.mutz@qt.io> | 2023-03-20 12:33:02 +0100 |
---|---|---|
committer | Marc Mutz <marc.mutz@qt.io> | 2023-06-12 13:11:26 +0000 |
commit | b209f943d2611fa4ac2dd9c64b1a014182b59a3d (patch) | |
tree | 744bb5708a5a52438aef7cba1ffb96d5c3a051eb /src/corelib/time/qdatetime.cpp | |
parent | e86e2752845ede635840851198c334f550fafb8f (diff) |
Replace {add,sub,mul}_overload with q{Add,Sub,Mul}Overload
These APIs started out as private APIs in qnumeric_p.h, but have since
been made pseudo-public in qnumeric.h. The qnumeric_p.h versions just
forward to the qnumeric.h ones, so just use the latter.
This is in preparation of removing the {add,sub,mul}_overflow
versions, which, despite being defined in the unnamed namespace, don't
sport the q prefix, so potentially clash with global symbols.
The change is a simple textual search and replace, manually excluding
qnumeric_p.h.
Picking to 6.5 to avoid cherry-pick conflicts going forward.
Pick-to: 6.6 6.5
Change-Id: Ic0f7c92f7c47923317109e8a9dc06fa66bdff2c2
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'src/corelib/time/qdatetime.cpp')
-rw-r--r-- | src/corelib/time/qdatetime.cpp | 26 |
1 files changed, 13 insertions, 13 deletions
diff --git a/src/corelib/time/qdatetime.cpp b/src/corelib/time/qdatetime.cpp index 55c9f120a1..cd4cdf7d5e 100644 --- a/src/corelib/time/qdatetime.cpp +++ b/src/corelib/time/qdatetime.cpp @@ -2583,8 +2583,8 @@ static QTime msecsToTime(qint64 msecs) // The inputs should not have opposite signs. static inline bool daysAndMillisOverflow(qint64 days, qint64 millisInDay, qint64 *sumMillis) { - return mul_overflow(days, std::integral_constant<qint64, MSECS_PER_DAY>(), sumMillis) - || add_overflow(*sumMillis, millisInDay, sumMillis); + return qMulOverflow(days, std::integral_constant<qint64, MSECS_PER_DAY>(), sumMillis) + || qAddOverflow(*sumMillis, millisInDay, sumMillis); } // Converts a date/time value into msecs @@ -2680,7 +2680,7 @@ QDateTimePrivate::ZoneState QDateTimePrivate::expressUtcAsLocal(qint64 utcMSecs) #if QT_CONFIG(timezone) // Use the system time-zone. if (const auto sys = QTimeZone::systemTimeZone(); sys.isValid()) { result.offset = sys.d->offsetFromUtc(utcMSecs); - if (add_overflow(utcMSecs, result.offset * MSECS_PER_SEC, &result.when)) + if (qAddOverflow(utcMSecs, result.offset * MSECS_PER_SEC, &result.when)) return result; result.dst = sys.d->isDaylightTime(utcMSecs) ? DaylightTime : StandardTime; result.valid = true; @@ -2697,15 +2697,15 @@ QDateTimePrivate::ZoneState QDateTimePrivate::expressUtcAsLocal(qint64 utcMSecs) const auto fakeJd = QGregorianCalendar::julianFromParts(systemTimeYearMatching(ymd.year), ymd.month, ymd.day); if (Q_UNLIKELY(!fakeJd - || mul_overflow(jd - *fakeJd, std::integral_constant<qint64, MSECS_PER_DAY>(), + || qMulOverflow(jd - *fakeJd, std::integral_constant<qint64, MSECS_PER_DAY>(), &diffMillis) - || sub_overflow(utcMSecs, diffMillis, &fakeUtc))) { + || qSubOverflow(utcMSecs, diffMillis, &fakeUtc))) { return result; } result = QLocalTime::utcToLocal(fakeUtc); // Now correct result.when for the use of the fake date: - if (!result.valid || add_overflow(result.when, diffMillis, &result.when)) { + if (!result.valid || qAddOverflow(result.when, diffMillis, &result.when)) { // If utcToLocal() failed, its return has the fake when; restore utcMSecs. // Fail on overflow, but preserve offset and DST-ness. result.when = utcMSecs; @@ -2784,7 +2784,7 @@ QDateTimePrivate::ZoneState QDateTimePrivate::localStateAtMillis(qint64 millis, auto result = QLocalTime::mapLocalTime(fake.shifted, dst); if (result.valid) { qint64 adjusted; - if (Q_UNLIKELY(add_overflow(result.when, millis - fake.shifted, &adjusted))) { + if (Q_UNLIKELY(qAddOverflow(result.when, millis - fake.shifted, &adjusted))) { using Bound = std::numeric_limits<qint64>; adjusted = millis < fake.shifted ? Bound::min() : Bound::max(); } @@ -4008,7 +4008,7 @@ void QDateTime::setMSecsSinceEpoch(qint64 msecs) if (QTimeZone::isUtcOrFixedOffset(spec)) { if (spec == Qt::OffsetFromUTC) state.offset = d->m_offsetFromUtc; - if (!state.offset || !add_overflow(msecs, state.offset * MSECS_PER_SEC, &state.when)) + if (!state.offset || !qAddOverflow(msecs, state.offset * MSECS_PER_SEC, &state.when)) status |= QDateTimePrivate::ValidityMask; } else if (spec == Qt::LocalTime) { state = QDateTimePrivate::expressUtcAsLocal(msecs); @@ -4021,7 +4021,7 @@ void QDateTime::setMSecsSinceEpoch(qint64 msecs) state.offset = data.offsetFromUtc; Q_ASSERT(state.offset >= -SECS_PER_DAY && state.offset <= SECS_PER_DAY); if (!state.offset - || !Q_UNLIKELY(add_overflow(msecs, state.offset * MSECS_PER_SEC, &state.when))) { + || !Q_UNLIKELY(qAddOverflow(msecs, state.offset * MSECS_PER_SEC, &state.when))) { d->m_status = mergeDaylightStatus(status | QDateTimePrivate::ValidityMask, data.daylightTimeOffset ? QDateTimePrivate::DaylightTime @@ -4062,7 +4062,7 @@ void QDateTime::setMSecsSinceEpoch(qint64 msecs) void QDateTime::setSecsSinceEpoch(qint64 secs) { qint64 msecs; - if (!mul_overflow(secs, std::integral_constant<qint64, MSECS_PER_SEC>(), &msecs)) { + if (!qMulOverflow(secs, std::integral_constant<qint64, MSECS_PER_SEC>(), &msecs)) { setMSecsSinceEpoch(msecs); } else if (d.isShort()) { d.data.status &= ~int(QDateTimePrivate::ValidityMask); @@ -4344,7 +4344,7 @@ QDateTime QDateTime::addYears(int nyears) const QDateTime QDateTime::addSecs(qint64 s) const { qint64 msecs; - if (mul_overflow(s, std::integral_constant<qint64, MSECS_PER_SEC>(), &msecs)) + if (qMulOverflow(s, std::integral_constant<qint64, MSECS_PER_SEC>(), &msecs)) return QDateTime(); return addMSecs(msecs); } @@ -4368,7 +4368,7 @@ QDateTime QDateTime::addMSecs(qint64 msecs) const case Qt::LocalTime: case Qt::TimeZone: // Convert to real UTC first in case this crosses a DST transition: - if (!add_overflow(toMSecsSinceEpoch(), msecs, &msecs)) { + if (!qAddOverflow(toMSecsSinceEpoch(), msecs, &msecs)) { dt.setMSecsSinceEpoch(msecs); } else if (dt.d.isShort()) { dt.d.data.status &= ~int(QDateTimePrivate::ValidityMask); @@ -4380,7 +4380,7 @@ QDateTime QDateTime::addMSecs(qint64 msecs) const case Qt::UTC: case Qt::OffsetFromUTC: // No need to convert, just add on - if (add_overflow(getMSecs(d), msecs, &msecs)) { + if (qAddOverflow(getMSecs(d), msecs, &msecs)) { if (dt.d.isShort()) { dt.d.data.status &= ~int(QDateTimePrivate::ValidityMask); } else { |