compile without foreach
Details
Details
Diff Detail
Diff Detail
- Repository
- R289 KNotifications
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
src/knotifyconfig.cpp | ||
---|---|---|
53–54 | How we can use iterator here ? |
src/notifybypopup.cpp | ||
---|---|---|
424 | this loop now will run forever... |
Comment Actions
This sadly breaks build with at least older versions of Phonon, as they have foreach loop in their public headers.
So this either needs a bump of the current
find_package(Phonon4Qt5 4.6.60 NO_MODULE)
or adding the QT_NO_FOREACH only if a newer working Phonon version is found.
Update: or removing the flag around the includes of Phonon for more hackery :)
Comment Actions
For the record, Laurent commented out the QT_NO_FOREACH define in commit cdfaff8ce6aa359555d0b611f9141d2012ec9dde, so the build should be fixed again.