diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2019-07-11 09:59:23 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2019-07-11 15:35:37 +0200 |
commit | 10c080521c525e1729b23dfa0afd08a1428f40d0 (patch) | |
tree | 851fa40f8dbee496826f0134a44e3dcd2a589a19 /src/plugins/directshow/directshow.pro | |
parent | 6dc5f327adfa987d2c03db3506c761d77a62e013 (diff) | |
download | qtmultimedia-10c080521c525e1729b23dfa0afd08a1428f40d0.tar.gz |
Eradicate the last Q_FOREACH and mark the module clean
This one is not trivial in that it isn't clear, a priori, that the
loop body doesn't re-enter the class and ends up modifying
m_videoProbes.
But MFVideoProbeControl::bufferProbed() just schedules a deferred call
to QMediaVideoProbeControl::videoFrameProbed() (via QueuedConnection),
so this cannot happen, because we end the loop before we re-enter the
event loop (if any) on this thread. Reentrancy from a different thread,
OTOH, is not possible because we're holding m_videoProbeMutex while
looping.
Change-Id: I6c1e22cd01112f0b2650e056f0b7f4a392370b84
Reviewed-by: VaL Doroshchuk <valentyn.doroshchuk@qt.io>
Diffstat (limited to 'src/plugins/directshow/directshow.pro')
0 files changed, 0 insertions, 0 deletions