In launcher/UnityApplications/launchermenu.cpp:
[...] if (folded) { /* Remove all actions but the title. */ for (int i = actions().size(); i > 0; --i) { QAction* action = actions().at(i - 1); [...]
The use of a reverse for loop does not seem necessary. A forward one would be simpler. And even better would be to use a Q_FOREACH.
« Back to merge proposal
In launcher/ UnityApplicatio ns/launchermenu .cpp:
[...]
QAction* action = actions().at(i - 1);
if (folded) {
/* Remove all actions but the title. */
for (int i = actions().size(); i > 0; --i) {
[...]
The use of a reverse for loop does not seem necessary. A forward one would be simpler. And even better would be to use a Q_FOREACH.