Inline QLocale's QString overloads via QStringView

Make declarations use same parameter name as implementation and
documentation, in the process. Fixed indent of some doc comments.

Change-Id: Ibaa3a5f6316a16c5b65f6ea6e0034f4eb2864b98
Reviewed-by: Mårten Nordheim <marten.nordheim@qt.io>
This commit is contained in:
Edward Welbourne 2020-11-09 15:43:29 +01:00
parent e71aafcb65
commit 7c28fa7717
2 changed files with 107 additions and 143 deletions

View File

@ -1397,6 +1397,8 @@ QString QLocale::scriptToString(QLocale::Script script)
#if QT_STRINGVIEW_LEVEL < 2
/*!
\fn short QLocale::toShort(const QString &s, bool *ok) const
Returns the short int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1409,12 +1411,9 @@ QString QLocale::scriptToString(QLocale::Script script)
\sa toUShort(), toString()
*/
short QLocale::toShort(const QString &s, bool *ok) const
{
return toIntegral_helper<short>(d, s, ok);
}
/*!
\fn ushort QLocale::toUShort(const QString &s, bool *ok) const
Returns the unsigned short int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1427,12 +1426,8 @@ short QLocale::toShort(const QString &s, bool *ok) const
\sa toShort(), toString()
*/
ushort QLocale::toUShort(const QString &s, bool *ok) const
{
return toIntegral_helper<ushort>(d, s, ok);
}
/*!
\fn int QLocale::toInt(const QString &s, bool *ok) const
Returns the int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1445,12 +1440,8 @@ ushort QLocale::toUShort(const QString &s, bool *ok) const
\sa toUInt(), toString()
*/
int QLocale::toInt(const QString &s, bool *ok) const
{
return toIntegral_helper<int>(d, s, ok);
}
/*!
\fn uint QLocale::toUInt(const QString &s, bool *ok) const
Returns the unsigned int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1463,12 +1454,10 @@ int QLocale::toInt(const QString &s, bool *ok) const
\sa toInt(), toString()
*/
uint QLocale::toUInt(const QString &s, bool *ok) const
{
return toIntegral_helper<uint>(d, s, ok);
}
/*!
\since 5.13
\fn long QLocale::toLong(const QString &s, bool *ok) const
Returns the long int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1479,17 +1468,12 @@ uint QLocale::toUInt(const QString &s, bool *ok) const
This function ignores leading and trailing whitespace.
\sa toInt(), toULong(), toDouble(), toString()
\since 5.13
*/
long QLocale::toLong(const QString &s, bool *ok) const
{
return toIntegral_helper<long>(d, s, ok);
}
*/
/*!
\since 5.13
\fn ulong QLocale::toULong(const QString &s, bool *ok) const
Returns the unsigned long int represented by the localized
string \a s.
@ -1501,16 +1485,10 @@ long QLocale::toLong(const QString &s, bool *ok) const
This function ignores leading and trailing whitespace.
\sa toLong(), toInt(), toDouble(), toString()
\since 5.13
*/
ulong QLocale::toULong(const QString &s, bool *ok) const
{
return toIntegral_helper<ulong>(d, s, ok);
}
/*!
\fn qlonglong QLocale::toLongLong(const QString &s, bool *ok) const
Returns the long long int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1523,13 +1501,9 @@ ulong QLocale::toULong(const QString &s, bool *ok) const
\sa toInt(), toULongLong(), toDouble(), toString()
*/
qlonglong QLocale::toLongLong(const QString &s, bool *ok) const
{
return toIntegral_helper<qlonglong>(d, s, ok);
}
/*!
\fn qulonglong QLocale::toULongLong(const QString &s, bool *ok) const
Returns the unsigned long long int represented by the localized
string \a s.
@ -1543,12 +1517,9 @@ qlonglong QLocale::toLongLong(const QString &s, bool *ok) const
\sa toLongLong(), toInt(), toDouble(), toString()
*/
qulonglong QLocale::toULongLong(const QString &s, bool *ok) const
{
return toIntegral_helper<qulonglong>(d, s, ok);
}
/*!
\fn float QLocale::toFloat(const QString &s, bool *ok) const
Returns the float represented by the localized string \a s.
Returns an infinity if the conversion overflows or 0.0 if the
@ -1565,12 +1536,8 @@ qulonglong QLocale::toULongLong(const QString &s, bool *ok) const
\sa toDouble(), toInt(), toString()
*/
float QLocale::toFloat(const QString &s, bool *ok) const
{
return QLocaleData::convertDoubleToFloat(toDouble(s, ok), ok);
}
/*!
\fn double QLocale::toDouble(const QString &s, bool *ok) const
Returns the double represented by the localized string \a s.
Returns an infinity if the conversion overflows or 0.0 if the
@ -1591,11 +1558,6 @@ float QLocale::toFloat(const QString &s, bool *ok) const
\sa toFloat(), toInt(), toString()
*/
double QLocale::toDouble(const QString &s, bool *ok) const
{
return d->m_data->stringToDouble(s, ok, d->m_numberOptions);
}
#endif // QT_STRINGVIEW_LEVEL < 2
/*!
@ -1679,6 +1641,7 @@ uint QLocale::toUInt(QStringView s, bool *ok) const
}
/*!
\since 5.13
Returns the long int represented by the localized string \a s.
If the conversion fails the function returns 0.
@ -1689,10 +1652,7 @@ uint QLocale::toUInt(QStringView s, bool *ok) const
This function ignores leading and trailing whitespace.
\sa toInt(), toULong(), toDouble(), toString()
\since 5.13
*/
*/
long QLocale::toLong(QStringView s, bool *ok) const
{
@ -1700,6 +1660,7 @@ long QLocale::toLong(QStringView s, bool *ok) const
}
/*!
\since 5.13
Returns the unsigned long int represented by the localized
string \a s.
@ -1711,9 +1672,7 @@ long QLocale::toLong(QStringView s, bool *ok) const
This function ignores leading and trailing whitespace.
\sa toLong(), toInt(), toDouble(), toString()
\since 5.13
*/
*/
ulong QLocale::toULong(QStringView s, bool *ok) const
{
@ -1852,8 +1811,32 @@ QString QLocale::toString(qulonglong i) const
QString QLocale::toString(QDate date, const QString &format) const
{
return QCalendar().dateTimeToString(format, QDateTime(), date, QTime(), *this);
return toString(date, qToStringViewIgnoringNull(format));
}
/*!
Returns a localized string representation of the given \a time according
to the specified \a format.
If \a format is an empty string, an empty string is returned.
\sa QTime::toString()
*/
QString QLocale::toString(QTime time, const QString &format) const
{
return toString(time, qToStringViewIgnoringNull(format));
}
/*!
\since 4.4
\fn QString QLocale::toString(const QDateTime &dateTime, const QString &format) const
Returns a localized string representation of the given \a dateTime according
to the specified \a format.
If \a format is an empty string, an empty string is returned.
\sa QDateTime::toString(), QDate::toString(), QTime::toString()
*/
#endif
/*!
@ -1915,20 +1898,6 @@ static bool timeFormatContainsAP(QStringView format)
return false;
}
#if QT_STRINGVIEW_LEVEL < 2
/*!
Returns a localized string representation of the given \a time according
to the specified \a format.
If \a format is an empty string, an empty string is returned.
\sa QTime::toString()
*/
QString QLocale::toString(QTime time, const QString &format) const
{
return QCalendar().dateTimeToString(format, QDateTime(), QDate(), time, *this);
}
#endif
/*!
\since 5.10
@ -1943,23 +1912,6 @@ QString QLocale::toString(QTime time, QStringView format) const
return QCalendar().dateTimeToString(format, QDateTime(), QDate(), time, *this);
}
#if QT_STRINGVIEW_LEVEL < 2
/*!
\since 4.4
Returns a localized string representation of the given \a dateTime according
to the specified \a format.
If \a format is an empty string, an empty string is returned.
\sa QDateTime::toString(), QDate::toString(), QTime::toString()
*/
QString QLocale::toString(const QDateTime &dateTime, const QString &format) const
{
return QCalendar().dateTimeToString(format, dateTime, QDate(), QTime(), *this);
}
#endif
/*!
\since 5.10

View File

@ -937,16 +937,26 @@ public:
QString nativeCountryName() const;
#if QT_STRINGVIEW_LEVEL < 2
short toShort(const QString &s, bool *ok = nullptr) const;
ushort toUShort(const QString &s, bool *ok = nullptr) const;
int toInt(const QString &s, bool *ok = nullptr) const;
uint toUInt(const QString &s, bool *ok = nullptr) const;
long toLong(const QString &s, bool *ok = nullptr) const;
ulong toULong(const QString &s, bool *ok = nullptr) const;
qlonglong toLongLong(const QString &s, bool *ok = nullptr) const;
qulonglong toULongLong(const QString &s, bool *ok = nullptr) const;
float toFloat(const QString &s, bool *ok = nullptr) const;
double toDouble(const QString &s, bool *ok = nullptr) const;
short toShort(const QString &s, bool *ok = nullptr) const
{ return toShort(qToStringViewIgnoringNull(s), ok); }
ushort toUShort(const QString &s, bool *ok = nullptr) const
{ return toUShort(qToStringViewIgnoringNull(s), ok); }
int toInt(const QString &s, bool *ok = nullptr) const
{ return toInt(qToStringViewIgnoringNull(s), ok); }
uint toUInt(const QString &s, bool *ok = nullptr) const
{ return toUInt(qToStringViewIgnoringNull(s), ok); }
long toLong(const QString &s, bool *ok = nullptr) const
{ return toLong(qToStringViewIgnoringNull(s), ok); }
ulong toULong(const QString &s, bool *ok = nullptr) const
{ return toULong(qToStringViewIgnoringNull(s), ok); }
qlonglong toLongLong(const QString &s, bool *ok = nullptr) const
{ return toLongLong(qToStringViewIgnoringNull(s), ok); }
qulonglong toULongLong(const QString &s, bool *ok = nullptr) const
{ return toULongLong(qToStringViewIgnoringNull(s), ok); }
float toFloat(const QString &s, bool *ok = nullptr) const
{ return toFloat(qToStringViewIgnoringNull(s), ok); }
double toDouble(const QString &s, bool *ok = nullptr) const
{ return toDouble(qToStringViewIgnoringNull(s), ok); }
#endif
short toShort(QStringView s, bool *ok = nullptr) const;
@ -973,12 +983,14 @@ public:
{ return toString(double(i), f, prec); }
#if QT_STRINGVIEW_LEVEL < 2
QString toString(QDate date, const QString &formatStr) const;
QString toString(QTime time, const QString &formatStr) const;
QString toString(const QDateTime &dateTime, const QString &format) const;
// (Can't inline first two: passing by value doesn't work when only forward-declared.)
QString toString(QDate date, const QString &format) const;
QString toString(QTime time, const QString &format) const;
QString toString(const QDateTime &dateTime, const QString &format) const
{ return toString(dateTime, qToStringViewIgnoringNull(format)); }
#endif
QString toString(QDate date, QStringView formatStr) const;
QString toString(QTime time, QStringView formatStr) const;
QString toString(QDate date, QStringView format) const;
QString toString(QTime time, QStringView format) const;
QString toString(const QDateTime &dateTime, QStringView format) const;
QString toString(QDate date, FormatType format = LongFormat) const;
QString toString(QTime time, FormatType format = LongFormat) const;
@ -987,10 +999,10 @@ public:
* after all tests we will remove non-calendar-aware version of these functions,
* and add a default value for both calendar instance, and format
*/
QString toString(QDate date, QStringView formatStr, QCalendar cal) const;
QString toString(QDate date, QStringView format, QCalendar cal) const;
QString toString(QDate date, FormatType format, QCalendar cal) const;
QString toString(const QDateTime &dateTime, FormatType format, QCalendar cal) const;
QString toString(const QDateTime &dateTime, QStringView formatStr, QCalendar cal) const;
QString toString(const QDateTime &dateTime, QStringView format, QCalendar cal) const;
QString dateFormat(FormatType format = LongFormat) const;
QString timeFormat(FormatType format = LongFormat) const;