From 7c1e0fef8e35ecd8487c41dc39e7ce46537f3040 Mon Sep 17 00:00:00 2001 From: Thiago Macieira Date: Thu, 24 May 2012 16:43:38 +0200 Subject: Gestures & events: compile with QT_STRICT_ITERATORS. There are a few mixed up iterators that needed cleaning up. We're missing a constUpperBound and constLowerBound function, though... This commit sneaks in one change to qtextformat related to QT_STRICT_ITERATORS but not to gestures and events. Change-Id: I8c7c840fb5f46c790adbf52952c6009c5b5f2f43 Reviewed-by: Denis Dzyubenko --- src/widgets/kernel/qgesturemanager.cpp | 37 +++++++++++++++++----------------- 1 file changed, 19 insertions(+), 18 deletions(-) (limited to 'src/widgets/kernel/qgesturemanager.cpp') diff --git a/src/widgets/kernel/qgesturemanager.cpp b/src/widgets/kernel/qgesturemanager.cpp index 8ba8904684..4d05e0da2c 100644 --- a/src/widgets/kernel/qgesturemanager.cpp +++ b/src/widgets/kernel/qgesturemanager.cpp @@ -140,8 +140,8 @@ void QGestureManager::unregisterGestureRecognizer(Qt::GestureType type) } } - QMap >::const_iterator iter = m_objectGestures.begin(); - while (iter != m_objectGestures.end()) { + QMap >::const_iterator iter = m_objectGestures.constBegin(); + while (iter != m_objectGestures.constEnd()) { ObjectGesture objectGesture = iter.key(); if (objectGesture.gesture == type) { foreach (QGesture *g, iter.value()) { @@ -248,9 +248,10 @@ bool QGestureManager::filterEventThroughContexts(const QMultiMap &const_recognizers = m_recognizers; QMap::const_iterator - typeToRecognizerIterator = m_recognizers.lowerBound(gestureType), - typeToRecognizerEnd = m_recognizers.upperBound(gestureType); + typeToRecognizerIterator = const_recognizers.lowerBound(gestureType), + typeToRecognizerEnd = const_recognizers.upperBound(gestureType); for (; typeToRecognizerIterator != typeToRecognizerEnd; ++typeToRecognizerIterator) { QGestureRecognizer *recognizer = typeToRecognizerIterator.value(); QObject *target = context.key(); @@ -468,8 +469,8 @@ bool QGestureManager::filterEvent(QWidget *receiver, QEvent *event) QWidget *w = receiver; typedef QMap::const_iterator ContextIterator; if (!w->d_func()->gestureContext.isEmpty()) { - for(ContextIterator it = w->d_func()->gestureContext.begin(), - e = w->d_func()->gestureContext.end(); it != e; ++it) { + for(ContextIterator it = w->d_func()->gestureContext.constBegin(), + e = w->d_func()->gestureContext.constEnd(); it != e; ++it) { types.insert(it.key(), 0); contexts.insertMulti(w, it.key()); } @@ -478,8 +479,8 @@ bool QGestureManager::filterEvent(QWidget *receiver, QEvent *event) w = w->isWindow() ? 0 : w->parentWidget(); while (w) { - for (ContextIterator it = w->d_func()->gestureContext.begin(), - e = w->d_func()->gestureContext.end(); it != e; ++it) { + for (ContextIterator it = w->d_func()->gestureContext.constBegin(), + e = w->d_func()->gestureContext.constEnd(); it != e; ++it) { if (!(it.value() & Qt::DontStartGestureOnChildren)) { if (!types.contains(it.key())) { types.insert(it.key(), 0); @@ -502,8 +503,8 @@ bool QGestureManager::filterEvent(QGraphicsObject *receiver, QEvent *event) QGraphicsObject *item = receiver; if (!item->QGraphicsItem::d_func()->gestureContext.isEmpty()) { typedef QMap::const_iterator ContextIterator; - for(ContextIterator it = item->QGraphicsItem::d_func()->gestureContext.begin(), - e = item->QGraphicsItem::d_func()->gestureContext.end(); it != e; ++it) { + for(ContextIterator it = item->QGraphicsItem::d_func()->gestureContext.constBegin(), + e = item->QGraphicsItem::d_func()->gestureContext.constEnd(); it != e; ++it) { types.insert(it.key(), 0); contexts.insertMulti(item, it.key()); } @@ -513,8 +514,8 @@ bool QGestureManager::filterEvent(QGraphicsObject *receiver, QEvent *event) while (item) { typedef QMap::const_iterator ContextIterator; - for (ContextIterator it = item->QGraphicsItem::d_func()->gestureContext.begin(), - e = item->QGraphicsItem::d_func()->gestureContext.end(); it != e; ++it) { + for (ContextIterator it = item->QGraphicsItem::d_func()->gestureContext.constBegin(), + e = item->QGraphicsItem::d_func()->gestureContext.constEnd(); it != e; ++it) { if (!(it.value() & Qt::DontStartGestureOnChildren)) { if (!types.contains(it.key())) { types.insert(it.key(), 0); @@ -559,8 +560,8 @@ void QGestureManager::getGestureTargets(const QSet &gestures, QWidget *w = widget->parentWidget(); while (w) { QMap::const_iterator it - = w->d_func()->gestureContext.find(type); - if (it != w->d_func()->gestureContext.end()) { + = w->d_func()->gestureContext.constFind(type); + if (it != w->d_func()->gestureContext.constEnd()) { // i.e. 'w' listens to gesture 'type' if (!(it.value() & Qt::DontStartGestureOnChildren) && w != widget) { // conflicting gesture! @@ -642,8 +643,8 @@ void QGestureManager::deliverEvents(const QSet &gestures, << "\n"; // if there are conflicting gestures, send the GestureOverride event - for (GesturesPerWidget::const_iterator it = conflictedGestures.begin(), - e = conflictedGestures.end(); it != e; ++it) { + for (GesturesPerWidget::const_iterator it = conflictedGestures.constBegin(), + e = conflictedGestures.constEnd(); it != e; ++it) { QWidget *receiver = it.key(); QList gestures = it.value(); DEBUG() << "QGestureManager::deliverEvents: sending GestureOverride to" @@ -676,8 +677,8 @@ void QGestureManager::deliverEvents(const QSet &gestures, } // delivering gestures that are not in conflicted state - for (GesturesPerWidget::const_iterator it = normalStartedGestures.begin(), - e = normalStartedGestures.end(); it != e; ++it) { + for (GesturesPerWidget::const_iterator it = normalStartedGestures.constBegin(), + e = normalStartedGestures.constEnd(); it != e; ++it) { if (!it.value().isEmpty()) { DEBUG() << "QGestureManager::deliverEvents: sending to" << it.key() << "gestures:" << it.value(); -- cgit v1.2.3