From fa6e82c412f604f0306004d7adda309d2d5d773b Mon Sep 17 00:00:00 2001 From: Lorn Potter Date: Tue, 8 Nov 2011 08:27:13 +1000 Subject: add more code coverage. qt_metacast was not tested. add Q_ASSERT instead of conditional checking for object. Change-Id: Ib8eaa0583d0f8f71b144941a34b1fe9109e85b7a Sanity-Review: Qt Sanity Bot Reviewed-by: Lincoln Ramsay --- src/sensors/gestures/qsensorgesture.cpp | 25 +++++++++++++------------ 1 file changed, 13 insertions(+), 12 deletions(-) (limited to 'src') diff --git a/src/sensors/gestures/qsensorgesture.cpp b/src/sensors/gestures/qsensorgesture.cpp index 228ee461..b5d7b7d6 100644 --- a/src/sensors/gestures/qsensorgesture.cpp +++ b/src/sensors/gestures/qsensorgesture.cpp @@ -152,19 +152,20 @@ void QSensorGesture::startDetection() return; Q_FOREACH (QSensorGestureRecognizer *recognizer, d_ptr->m_sensorRecognizers) { - if (recognizer !=0) { //it shouldn't be, but I am paranoid - connect(recognizer,SIGNAL(detected(QString)), - this,SIGNAL(detected(QString)),Qt::UniqueConnection); - - //connect recognizer signals - Q_FOREACH (QString method, recognizer->gestureSignals()) { - method.prepend(QLatin1String("2")); - connect(recognizer, method.toLatin1(), - this, method.toLatin1(), Qt::UniqueConnection); - } - - recognizer->startBackend(); + + Q_ASSERT(recognizer !=0); + + connect(recognizer,SIGNAL(detected(QString)), + this,SIGNAL(detected(QString)),Qt::UniqueConnection); + + //connect recognizer signals + Q_FOREACH (QString method, recognizer->gestureSignals()) { + method.prepend(QLatin1String("2")); + connect(recognizer, method.toLatin1(), + this, method.toLatin1(), Qt::UniqueConnection); } + + recognizer->startBackend(); } d_ptr->isActive = true; } -- cgit v1.2.3