summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarc Mutz <marc.mutz@qt.io>2022-01-21 10:08:37 +0100
committerMarc Mutz <marc.mutz@qt.io>2022-01-22 04:12:48 +0100
commit31a39bd55da1635742ef7d4324e980c2a6b5b641 (patch)
tree8b9d9cd7349ed1e0c0d8479ec814f373aeab46f6
parent806ba27419f27bb5264c252a0d8faac7c3854b19 (diff)
QByteArrayList: simplify the join() overload set already now
... instead of waiting for Qt 7. Found in API review. [ChangeLog][QtCore][Potentially Source-Incompatible Changes] [QByteArrayList] The join() overload set has changed. Code such as qOverload<>(&QByteArrayList::join) will have to be rewritten, e.g. using lambdas. We advise against taking addresses of library functions other than signals and slots. Pick-to: 6.3 Change-Id: I67449df9adc2efea7f1163034caa135f31f39e7c Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
-rw-r--r--src/corelib/text/qbytearraylist.cpp8
-rw-r--r--src/corelib/text/qbytearraylist.h4
2 files changed, 2 insertions, 10 deletions
diff --git a/src/corelib/text/qbytearraylist.cpp b/src/corelib/text/qbytearraylist.cpp
index cd77cd905a..e23bdc7bd8 100644
--- a/src/corelib/text/qbytearraylist.cpp
+++ b/src/corelib/text/qbytearraylist.cpp
@@ -103,12 +103,6 @@ QT_BEGIN_NAMESPACE
*/
/*!
- \fn QByteArray QByteArrayList::join() const
-
- Joins all the byte arrays into a single byte array.
-*/
-
-/*!
\fn QByteArray QByteArrayList::join(const QByteArray &separator) const
Joins all the byte arrays into a single byte array with each
@@ -120,7 +114,7 @@ QT_BEGIN_NAMESPACE
\since 6.3
Joins all the byte arrays into a single byte array with each
- element separated by the given \a separator.
+ element separated by the given \a separator, if any.
*/
/*!
diff --git a/src/corelib/text/qbytearraylist.h b/src/corelib/text/qbytearraylist.h
index 890cbcef2d..ce613fbe3f 100644
--- a/src/corelib/text/qbytearraylist.h
+++ b/src/corelib/text/qbytearraylist.h
@@ -80,9 +80,7 @@ public:
using QListSpecialMethodsBase<QByteArray>::lastIndexOf;
using QListSpecialMethodsBase<QByteArray>::contains;
- inline QByteArray join() const
- { return join(QByteArrayView{}); }
- inline QByteArray join(QByteArrayView sep) const // ### Qt 7: merge with the () overload
+ QByteArray join(QByteArrayView sep = {}) const
{
return QtPrivate::QByteArrayList_join(self(), sep);
}