-
Task
-
Resolution: Unresolved
-
P2: Important
-
None
-
None
-
None
-
21
Let's be honest: the change from int to qsizetype sizes in Qt 6.0 was more than rushed. Half the API and a good chunk of implementation code is still using int when it should be using qsizetype, and we have been papering over this issue by making qMin() allow mixed-type arguments and, apparently, by removing the annoying MSVC warnings that, in Qt 5, so nicely pin-pointed all these implicit conversions as hard errors.
There were several real bugs in e.g. the QBuffer code corresponding to int/qsizetype, so there's probably more.
Acceptance criterium: every int has been inspected and, if needed, ported to qsizetype. Unfortunately, relying on compiler warnings isn't enough, as explicit casts won't raise warnings, but need to be ported nonetheless.
| For Gerrit Dashboard: QTBUG-103527 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 425059,5 | Port qenvironmentvariables.cpp to qsizetype | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 425140,3 | Port qenvironmentvariables.cpp to qsizetype | 6.4 | qt/qtbase | Status: MERGED | +2 | 0 |
| 425141,3 | Port qenvironmentvariables.cpp to qsizetype | 6.3 | qt/qtbase | Status: MERGED | +2 | 0 |
| 425145,3 | Port qenvironmentvariables.cpp to qsizetype | tqtc/lts-6.2 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |
| 439972,3 | qlogging: use qsizetype to fix Wconversion warnings | dev | qt/qtbase | Status: MERGED | +2 | 0 |
| 440124,2 | qlogging: use qsizetype to fix Wconversion warnings | 6.4 | qt/qtbase | Status: MERGED | +2 | 0 |
| 440128,3 | qlogging: use qsizetype to fix Wconversion warnings | tqtc/lts-6.2 | qt/tqtc-qtbase | Status: MERGED | +2 | 0 |