Details
-
Bug
-
Resolution: Unresolved
-
P2: Important
-
None
-
6.7
-
None
-
8
-
a6d2e45d5 (dev), 86e180ec4 (6.7), db8901c04 (tqtc/lts-6.5), a6f01f12c (tqtc/lts-6.2), 4bb36a6e7 (dev), 3750611d6 (6.7), 3c0bd5c36 (tqtc/lts-6.5)
-
Foundation Sprint 108, Foundation Sprint 109, Foundation Sprint 110, Foundation Sprint 111, Foundation Sprint 112, Foundation Sprint 113, Foundation Sprint 114, Foundation Sprint 115, Foundation Sprint 116, Foundation Sprint 117, Foundation Sprint 118, Foundation Sprint 119, Foundation Sprint 120
Description
QString fmt = "%1"; assert(fmt.arg(u' ') == " "); // FAIL: result is "32"
Problem here is that we have a large historic body of code which may rely on this. Maybe we need to = delete the overload?
The QLatin1StringView and QStringView overloads use the multi-arg code path for this, and treat char16_t as a character.
This also means that the behaviour differs between QString::arg() and QString::multiArg().
Attachments
Issue Links
- resulted from
-
QTBUG-124365 Port QString::arg() to QAnyStringView
-
- In Review
-
Gerrit Reviews
| For Gerrit Dashboard: QTBUG-125588 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 564066,10 | QString: toward UTF-8 arg() support [3½/4]: port unary arg() to QAnyStringView | dev | qt/qtbase | Status: NEW | 0 | -1 |
| 562942,8 | tst_QStringApiSymmetry: add tests for unary arg() calls | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 564018,3 | tst_QString: add more checks for unary arg() calls | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 564136,2 | tst_QString: add more checks for unary arg() calls | 6.7 | qt/qtbase | Status: MERGED | +2 | 0 |
| 564159,2 | tst_QString: add more checks for unary arg() calls | tqtc/lts-6.5 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |
| 564253,2 | tst_QString: add more checks for unary arg() calls | tqtc/lts-6.2 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |
| 564440,2 | tst_QStringApiSymmetry: add tests for unary arg() calls | 6.7 | qt/qtbase | Status: MERGED | +2 | 0 |
| 564474,2 | tst_QStringApiSymmetry: add tests for unary arg() calls | tqtc/lts-6.5 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |