Symbian QAudioOutput::suspend() was resetting processedUSecs() to zero

Because of the logic in the Symbian implementations of suspend()
and processedUSecs(), the behaviour prior to this patch was as follows:
- after calling suspend(), processedUSecs() reported zero
- when resume() is called, the amount of silent padding data inserted into
the output stream is equal to the amount of data played prior to suspend()

The patch corrects the above defect, but introduces a new one, which is
described in QTBUG-9322

Reviewed-by: trustme

1 files changed, 2 insertions, 1 deletions

diff --git a/src/multimedia/audio/qaudiooutput_symbian_p.cpp b/src/multimedia/audio/qaudiooutput_symbian_p.cpp
index 945a08d..3f8e933 100644
--- a/src/multimedia/audio/qaudiooutput_symbian_p.cpp
+++ b/src/multimedia/audio/qaudiooutput_symbian_p.cpp
@@ -185,10 +185,11 @@ void QAudioOutputPrivate::suspend()
 
         const qint64 samplesWritten = SymbianAudio::Utils::bytesToSamples(
                                           m_format, m_bytesWritten);
-        m_bytesWritten = 0;
 
         const qint64 samplesPlayed = getSamplesPlayed();
 
+        m_bytesWritten = 0;
+
         // CMMFDevSound::Pause() is not guaranteed to work correctly in all
         // implementations, for play-mode DevSound sessions.  We therefore
         // have to implement suspend() by calling CMMFDevSound::Stop().