diff --git a/src/kactioncollection.cpp b/src/kactioncollection.cpp --- a/src/kactioncollection.cpp +++ b/src/kactioncollection.cpp @@ -767,16 +767,16 @@ signal.methodSignature() == "actionHovered(QAction*)") { if (!d->connectHovered) { d->connectHovered = true; - Q_FOREACH (QAction *action, actions()) { + for (QAction *action : qAsConst(d->actions)) { connect(action, &QAction::hovered, this, &KActionCollection::slotActionHovered); } } } else if (signal.methodSignature() == "actionTriggered(QAction*)") { if (!d->connectTriggered) { d->connectTriggered = true; - Q_FOREACH (QAction *action, actions()) { + for (QAction *action: qAsConst(d->actions)) { connect(action, &QAction::triggered, this, &KActionCollection::slotActionTriggered); } @@ -793,7 +793,7 @@ void KActionCollection::associateWidget(QWidget *widget) const { - Q_FOREACH (QAction *action, actions()) { + for (QAction *action : qAsConst(d->actions)) { if (!widget->actions().contains(action)) { widget->addAction(action); } @@ -812,7 +812,7 @@ void KActionCollection::removeAssociatedWidget(QWidget *widget) { - Q_FOREACH (QAction *action, actions()) { + for (QAction *action : qAsConst(d->actions)) { widget->removeAction(action); } @@ -862,8 +862,7 @@ void KActionCollection::clearAssociatedWidgets() { for (QWidget *widget : qAsConst(d->associatedWidgets)) { - const auto actions = this->actions(); - for (QAction *action : actions) { + for (QAction *action : qAsConst(d->actions)) { widget->removeAction(action); } }