summaryrefslogtreecommitdiffstats
path: root/src/corelib/compat
diff options
context:
space:
mode:
authorIvan Solovev <ivan.solovev@qt.io>2024-01-17 16:30:03 +0100
committerIvan Solovev <ivan.solovev@qt.io>2024-01-22 14:18:09 +0100
commitcde7d5d3ff56fdf219771c65bf93f3f08757f154 (patch)
treee57f3a91a9af96b9544250adb0756ec1d3afcb13 /src/corelib/compat
parente957ed97e0a7f6974db7d9f7ea52d94785464880 (diff)
Make public QLocale's constant for the two-digit-year base
The private QLocale::FirstTwoDigitYear constant, which is used as a default value for the parameters in some methods, causes troubles for Qt for Python Team, because they need to use the default values in the binding code. It also potentially create some inconveniences for the users who want to write functions wrapping these methods. The name is also confusing, because, when read out of context, it implies that there might be SecondTwoDigitYear, etc... Rename QLocale::FirstTwoDigitYear to QLocale::DefaultTwoDigitBaseYear and make it public. Now when the constant is public, we can use it in QDate, instead of introducing another constant, so do that. The qdatetime.h header already includes qlocale.h via qcalendar.h, but, rather than relying on this transitive include, add it explicitly. As pointed out by Thiago [1], the static constexpr members of exported classes need out-of-line definitions, so add such definition to qlocale.cpp. Amends 41f84f3ddb780ec751e3fc706dd242fc4a99de7a Found in 6.7 API review. [1]: https://lists.qt-project.org/pipermail/development/2024-January/044888.html Pick-to: 6.7 Change-Id: Ib3c6f1d5b181968bf311fd0435173e025a369865 Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io> Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io> Reviewed-by: Marc Mutz <marc.mutz@qt.io>
Diffstat (limited to 'src/corelib/compat')
-rw-r--r--src/corelib/compat/removed_api.cpp16
1 files changed, 8 insertions, 8 deletions
diff --git a/src/corelib/compat/removed_api.cpp b/src/corelib/compat/removed_api.cpp
index 87b2d14916..0b2dfb57fb 100644
--- a/src/corelib/compat/removed_api.cpp
+++ b/src/corelib/compat/removed_api.cpp
@@ -794,42 +794,42 @@ QString QLocale::bcp47Name() const
QDate QLocale::toDate(const QString &string, FormatType format) const
{
- return toDate(string, dateFormat(format), FirstTwoDigitYear);
+ return toDate(string, dateFormat(format), DefaultTwoDigitBaseYear);
}
QDate QLocale::toDate(const QString &string, FormatType format, QCalendar cal) const
{
- return toDate(string, dateFormat(format), cal, FirstTwoDigitYear);
+ return toDate(string, dateFormat(format), cal, DefaultTwoDigitBaseYear);
}
QDateTime QLocale::toDateTime(const QString &string, FormatType format) const
{
- return toDateTime(string, dateTimeFormat(format), FirstTwoDigitYear);
+ return toDateTime(string, dateTimeFormat(format), DefaultTwoDigitBaseYear);
}
QDateTime QLocale::toDateTime(const QString &string, FormatType format, QCalendar cal) const
{
- return toDateTime(string, dateTimeFormat(format), cal, FirstTwoDigitYear);
+ return toDateTime(string, dateTimeFormat(format), cal, DefaultTwoDigitBaseYear);
}
QDate QLocale::toDate(const QString &string, const QString &format) const
{
- return toDate(string, format, QCalendar(), FirstTwoDigitYear);
+ return toDate(string, format, QCalendar(), DefaultTwoDigitBaseYear);
}
QDate QLocale::toDate(const QString &string, const QString &format, QCalendar cal) const
{
- return toDate(string, format, cal, FirstTwoDigitYear);
+ return toDate(string, format, cal, DefaultTwoDigitBaseYear);
}
QDateTime QLocale::toDateTime(const QString &string, const QString &format) const
{
- return toDateTime(string, format, QCalendar(), FirstTwoDigitYear);
+ return toDateTime(string, format, QCalendar(), DefaultTwoDigitBaseYear);
}
QDateTime QLocale::toDateTime(const QString &string, const QString &format, QCalendar cal) const
{
- return toDateTime(string, format, cal, FirstTwoDigitYear);
+ return toDateTime(string, format, cal, DefaultTwoDigitBaseYear);
}
#include "qobject.h"