diff options
Diffstat (limited to 'src/designer/src/lib/shared/actionprovider_p.h')
-rw-r--r-- | src/designer/src/lib/shared/actionprovider_p.h | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/designer/src/lib/shared/actionprovider_p.h b/src/designer/src/lib/shared/actionprovider_p.h index 63d902951..a608781c3 100644 --- a/src/designer/src/lib/shared/actionprovider_p.h +++ b/src/designer/src/lib/shared/actionprovider_p.h @@ -67,15 +67,14 @@ template <class Widget> int actionIndexAt(const Widget *w, const QPoint &pos, Qt::Orientation orientation) { const auto actions = w->actions(); - const int actionCount = actions.count(); - if (actionCount == 0) + if (actions.isEmpty()) return -1; // actionGeometry() can be wrong sometimes; it returns a geometry that // stretches to the end of the toolbar/menu bar. So, check from the beginning // in the case of a horizontal right-to-left orientation. const bool checkTopRight = orientation == Qt::Horizontal && w->layoutDirection() == Qt::RightToLeft; const QPoint topRight = QPoint(w->rect().width(), 0); - for (int index = 0; index < actionCount; ++index) { + for (qsizetype index = 0, actionCount = actions.size(); index < actionCount; ++index) { QRect g = w->actionGeometry(actions.at(index)); if (checkTopRight) g.setTopRight(topRight); @@ -83,7 +82,7 @@ template <class Widget> g.setTopLeft(QPoint(0, 0)); if (g.contains(pos)) - return index; + return int(index); } return -1; } |