summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFriedemann Kleint <Friedemann.Kleint@qt.io>2017-06-20 12:55:01 +0200
committerFriedemann Kleint <Friedemann.Kleint@qt.io>2017-06-20 12:03:04 +0000
commit523494be2dd799eb838ef4724de77111ce77c8f3 (patch)
tree2850ca4f19cac06c5fdda2a529f8f44a0bad9ce7 /src
parent14c884d9cfb97d889af1ae17da903890e519275c (diff)
DirectShowPlayerService::doSetVideoProbe(): Break out of loop
Fix g++ 7.1 warning: player\directshowplayerservice.cpp:724:9: warning: this 'if' clause does not guard... [-Wmisleading-indentation] if (SUCCEEDED(DirectShowUtils::connectFilters(m_graph, m_source, m_videoSampleGrabber->filter(), true))) player\directshowplayerservice.cpp:726:13: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if' break; Amends 8e4d966f4e5546787257de046d2c3af8a19214b4. Change-Id: I491e14e0116140d2ebb870441c2f048a2d7ddc31 Reviewed-by: Christian Stromme <christian.stromme@qt.io>
Diffstat (limited to 'src')
-rw-r--r--src/plugins/directshow/player/directshowplayerservice.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/plugins/directshow/player/directshowplayerservice.cpp b/src/plugins/directshow/player/directshowplayerservice.cpp
index 905e03df..2f959b23 100644
--- a/src/plugins/directshow/player/directshowplayerservice.cpp
+++ b/src/plugins/directshow/player/directshowplayerservice.cpp
@@ -724,9 +724,10 @@ void DirectShowPlayerService::doSetVideoProbe(QMutexLocker *locker)
for (int i = 0; i != items; ++i) {
mediaType->subtype = subtypes[i];
m_videoSampleGrabber->setMediaType(&mediaType);
- if (SUCCEEDED(DirectShowUtils::connectFilters(m_graph, m_source, m_videoSampleGrabber->filter(), true)))
+ if (SUCCEEDED(DirectShowUtils::connectFilters(m_graph, m_source, m_videoSampleGrabber->filter(), true))) {
connected = true;
break;
+ }
}
if (!connected) {