Compile without Q_FOREACH
Needs ReviewPublic

Authored by tcanabrava on May 10 2019, 12:25 PM.

Details

Reviewers
hindenburg
sandsmark
Group Reviewers
Konsole

Diff Detail

Repository
R319 Konsole
Branch
removeForeach
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 11722
Build 11740: arc lint + arc unit
tcanabrava created this revision.May 10 2019, 12:25 PM
Restricted Application added a project: Konsole. · View Herald TranscriptMay 10 2019, 12:25 PM
Restricted Application added a subscriber: konsole-devel. · View Herald Transcript
tcanabrava requested review of this revision.May 10 2019, 12:25 PM

Thanks for doing this - I was going to eventually get around to it.

Note that this generates a lot of these warnings

warning: c++11 range-loop might detach Qt container (QList) [-Wclazy-range-loop]

I’ll look the warnings, none of them appear on Gcc. Should be a clazy
warning

Em sáb, 11 de mai de 2019 às 16:28, Kurt Hindenburg <
noreply@phabricator.kde.org> escreveu:

hindenburg added a comment. View Revision
https://phabricator.kde.org/D21123

Thanks for doing this - I was going to eventually get around to it.

Note that this generates a lot of these warnings

warning: c++11 range-loop might detach Qt container (QList)
[-Wclazy-range-loop]

*REPOSITORY*
R319 Konsole

*REVISION DETAIL*
https://phabricator.kde.org/D21123

*To: *tcanabrava, Konsole, hindenburg, sandsmark
*Cc: *konsole-devel, gennad, thsurrel, ngraham, maximilianocuria,
hindenburg