- Q_FOREACH (Application* application, m_applications) { + for (Application* application : m_applications) { not a blocker, just a hint the original *might* be better. Ref: http://www.dvratil.cz/2015/06/qt-containers-and-c11-range-based-loops/
Other than that, code looks ok
« Back to merge proposal
- Q_FOREACH (Application* application, m_applications) { www.dvratil. cz/2015/ 06/qt-container s-and-c11- range-based- loops/
+ for (Application* application : m_applications) {
not a blocker, just a hint the original *might* be better. Ref:
http://
Other than that, code looks ok