Details
-
Task
-
Resolution: Fixed
-
P3: Somewhat important
-
None
-
None
-
3386c4497 (dev), 0ff112a78 (dev), eb3cdb90d (6.9), 563214602 (6.8)
Description
See epic for details.
At the time of writing (dev = 6.7), there are only tests that still use foreach/Q_FOREACH.
Acceptance criteria:
- Q_FOREACH/foreach are ported away from (see porting guide in epic before attempting anything!)
- the #undef QT_NO_FOREACH are gone
- files that contained them are removed from NO_PCH_SOURCES
- (note that one file in included in two projects, and therefore two NO_PCH_SOURCES)
- files that contained them are removed from NO_PCH_SOURCES
Attachments
Issue Links
- is cloned by
-
-
- Closed
-
| For Gerrit Dashboard: QTBUG-115805 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 494757,3 | Mark the module as free of Q_FOREACH, except where it isn't | dev | qt/qtwebengine | Status: MERGED | +2 | 0 |
| 624492,4 | Port away from foreach | dev | qt/qtwebengine | Status: MERGED | +2 | 0 |
| 625696,2 | Port away from foreach | 6.9 | qt/qtwebengine | Status: MERGED | +2 | 0 |
| 625925,6 | Port away from foreach | 6.8 | qt/qtwebengine | Status: MERGED | +2 | 0 |