summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGareth Stockwell <ext-gareth.stockwell@nokia.com>2010-01-08 15:07:36 (GMT)
committerJason McDonald <jason.mcdonald@nokia.com>2010-01-11 13:45:36 (GMT)
commitb2ee3b3e114a126e2452260266539d724cb7a38a (patch)
treeec9a5012368cb637a97359b7a713f59feb1cb451
parenteb2d09cff261b6e2fdff7b45b0cd30151c71bc97 (diff)
downloadQt-b2ee3b3e114a126e2452260266539d724cb7a38a.zip
Qt-b2ee3b3e114a126e2452260266539d724cb7a38a.tar.gz
Qt-b2ee3b3e114a126e2452260266539d724cb7a38a.tar.bz2
Fixed build breakage on S60 3.1 due to audio effects changes
Removed calls to the video overload of the audio effect constructors, e.g. CAudioEqualizer::NewL(VideoPlayerUtility&) Task-number: QTBUG-7223 Reviewed-by: trustme (cherry picked from commit a3e6a04448979aaa6ada7aa434de3137f6cf0563)
-rw-r--r--src/3rdparty/phonon/mmf/abstractaudioeffect.cpp11
-rw-r--r--src/3rdparty/phonon/mmf/abstractaudioeffect.h1
-rw-r--r--src/3rdparty/phonon/mmf/audioequalizer.cpp20
-rw-r--r--src/3rdparty/phonon/mmf/audioequalizer.h1
-rw-r--r--src/3rdparty/phonon/mmf/bassboost.cpp7
-rw-r--r--src/3rdparty/phonon/mmf/bassboost.h1
6 files changed, 10 insertions, 31 deletions
diff --git a/src/3rdparty/phonon/mmf/abstractaudioeffect.cpp b/src/3rdparty/phonon/mmf/abstractaudioeffect.cpp
index 8c73027..e7ef9b2 100644
--- a/src/3rdparty/phonon/mmf/abstractaudioeffect.cpp
+++ b/src/3rdparty/phonon/mmf/abstractaudioeffect.cpp
@@ -80,15 +80,10 @@ void AbstractAudioEffect::connectMediaObject(MediaObject *mediaObject)
if (AudioPlayer *audioPlayer = qobject_cast<AudioPlayer *>(player)) {
connectAudioPlayer(audioPlayer->nativePlayer());
- } else {
- VideoPlayer *videoPlayer = qobject_cast<VideoPlayer *>(player);
- Q_ASSERT_X(videoPlayer, Q_FUNC_INFO, "Player type not recognised");
- connectVideoPlayer(videoPlayer->nativePlayer());
+ applyParameters();
+ // TODO: handle audio effect errors
+ TRAP_IGNORE(m_effect->EnableL());
}
-
- applyParameters();
- // TODO: handle audio effect errors
- TRAP_IGNORE(m_effect->EnableL());
}
}
diff --git a/src/3rdparty/phonon/mmf/abstractaudioeffect.h b/src/3rdparty/phonon/mmf/abstractaudioeffect.h
index 10578af..6f74a73 100644
--- a/src/3rdparty/phonon/mmf/abstractaudioeffect.h
+++ b/src/3rdparty/phonon/mmf/abstractaudioeffect.h
@@ -86,7 +86,6 @@ protected:
void disconnectMediaObject(MediaObject *mediaObject);
virtual void connectAudioPlayer(AudioPlayer::NativePlayer *player) = 0;
- virtual void connectVideoPlayer(VideoPlayer::NativePlayer *player) = 0;
virtual void applyParameters() = 0;
virtual void parameterChanged(const int id,
diff --git a/src/3rdparty/phonon/mmf/audioequalizer.cpp b/src/3rdparty/phonon/mmf/audioequalizer.cpp
index 51f1c32..c2936c5 100644
--- a/src/3rdparty/phonon/mmf/audioequalizer.cpp
+++ b/src/3rdparty/phonon/mmf/audioequalizer.cpp
@@ -49,21 +49,15 @@ void AudioEqualizer::connectAudioPlayer(AudioPlayer::NativePlayer *player)
m_effect.reset(ptr);
}
-void AudioEqualizer::connectVideoPlayer(VideoPlayer::NativePlayer *player)
-{
- CAudioEqualizer *ptr = 0;
- QT_TRAP_THROWING(ptr = CAudioEqualizer::NewL(*player));
- m_effect.reset(ptr);
-}
-
void AudioEqualizer::applyParameters()
{
- Q_ASSERT_X(m_effect.data(), Q_FUNC_INFO, "Effect not created");
- EffectParameter param;
- foreach (param, parameters()) {
- const int band = param.id();
- const int level = parameterValue(param).toInt();
- setBandLevel(band, level);
+ if (m_effect.data()) {
+ EffectParameter param;
+ foreach (param, parameters()) {
+ const int band = param.id();
+ const int level = parameterValue(param).toInt();
+ setBandLevel(band, level);
+ }
}
}
diff --git a/src/3rdparty/phonon/mmf/audioequalizer.h b/src/3rdparty/phonon/mmf/audioequalizer.h
index 9910ea4..10fe9ad 100644
--- a/src/3rdparty/phonon/mmf/audioequalizer.h
+++ b/src/3rdparty/phonon/mmf/audioequalizer.h
@@ -44,7 +44,6 @@ public:
protected:
// AbstractAudioEffect
virtual void connectAudioPlayer(AudioPlayer::NativePlayer *player);
- virtual void connectVideoPlayer(VideoPlayer::NativePlayer *player);
virtual void applyParameters();
virtual void parameterChanged(const int id, const QVariant &value);
diff --git a/src/3rdparty/phonon/mmf/bassboost.cpp b/src/3rdparty/phonon/mmf/bassboost.cpp
index 36069fb..ae96b45 100644
--- a/src/3rdparty/phonon/mmf/bassboost.cpp
+++ b/src/3rdparty/phonon/mmf/bassboost.cpp
@@ -46,13 +46,6 @@ void BassBoost::connectAudioPlayer(AudioPlayer::NativePlayer *player)
m_effect.reset(ptr);
}
-void BassBoost::connectVideoPlayer(VideoPlayer::NativePlayer *player)
-{
- CBassBoost *ptr = 0;
- QT_TRAP_THROWING(ptr = CBassBoost::NewL(*player));
- m_effect.reset(ptr);
-}
-
void BassBoost::applyParameters()
{
// No parameters to apply
diff --git a/src/3rdparty/phonon/mmf/bassboost.h b/src/3rdparty/phonon/mmf/bassboost.h
index 1b893db..4ad0a6c 100644
--- a/src/3rdparty/phonon/mmf/bassboost.h
+++ b/src/3rdparty/phonon/mmf/bassboost.h
@@ -42,7 +42,6 @@ public:
protected:
// AbstractAudioEffect
virtual void connectAudioPlayer(AudioPlayer::NativePlayer *player);
- virtual void connectVideoPlayer(VideoPlayer::NativePlayer *player);
virtual void applyParameters();
virtual void parameterChanged(const int id, const QVariant &value);