Details
-
Epic
-
Resolution: Unresolved
-
P2: Important
-
None
-
None
-
QStringList-6.7
-
e1dcc858b (dev), e1fe8963a (6.6), ea5035bfb (tqtc/lts-6.5), 0d3585447 (tqtc/lts-6.2), ae62bd02f (tqtc/lts-5.15)
Description
QStringList has several shortcomings. This epic sketches them. When grooming, please create proper tasks for each item under this epic:
In no particular order:
- the docs show only a subset of functions
- e.g. indexOf only for QRegularExpression, not for QString, QStringView, QLatin1StringView (which partly come in via the indexOf template function, but it looks odd nonetheless)
- doc fixes should be backported to current LTS, at least, so may need a Q_QDOC-block to fake them before actually implementing them (which is for dev only, because it adds new symbols)
- the following functions are missing (some) overloads (join and contains are considered complete, for now):
- indexOf/lastIndexOf
- they work because they have a template backend, but fact this is visible in Clang -ftime-trace (→ QTBUG-97601), so they should get the same treatment as contains (out-of-line QStringList_xxx functions
- filter() (QL1SV)
- indexOf/lastIndexOf
- replaceInStrings() should be QAnyStringView-ified (two arguments = too many overloads)
- the following functions could do with a CaseSensitivity argument:
- indexOf/lastIndexOf (cf. contains())
- must take care where to place it w.r.t. the from parameter
- removeDuplicates()?
- indexOf/lastIndexOf (cf. contains())
Attachments
1.
|
QStringList: add missing API docs | Reported | Ahmad Samir |