From 719b9738d8292479c3af296b50a42e82d96ba538 Mon Sep 17 00:00:00 2001 From: Christian Muehlhaeuser Date: Thu, 3 Jan 2013 20:00:13 +0100 Subject: [PATCH] * Make AudioContols directly connect to AudioEngine's repeat & shuffle signals. --- src/AudioControls.cpp | 10 ++++++---- src/TomahawkWindow.cpp | 7 +------ 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/src/AudioControls.cpp b/src/AudioControls.cpp index dd8d82e42..c92219994 100644 --- a/src/AudioControls.cpp +++ b/src/AudioControls.cpp @@ -140,6 +140,8 @@ AudioControls::AudioControls( QWidget* parent ) connect( AudioEngine::instance(), SIGNAL( timerMilliSeconds( qint64 ) ), SLOT( onPlaybackTimer( qint64 ) ) ); connect( AudioEngine::instance(), SIGNAL( volumeChanged( int ) ), SLOT( onVolumeChanged( int ) ) ); connect( AudioEngine::instance(), SIGNAL( controlStateChanged() ), SLOT( onControlStateChanged() ) ); + connect( AudioEngine::instance(), SIGNAL( repeatModeChanged( Tomahawk::PlaylistModes::RepeatMode ) ), SLOT( onRepeatModeChanged( Tomahawk::PlaylistModes::RepeatMode ) ) ); + connect( AudioEngine::instance(), SIGNAL( shuffleModeChanged( bool ) ), SLOT( onShuffleModeChanged( bool ) ) ); ui->buttonAreaLayout->setSpacing( 0 ); ui->stackedLayout->setSpacing( 0 ); @@ -560,21 +562,21 @@ AudioControls::onRepeatClicked() case PlaylistModes::NoRepeat: { // switch to RepeatOne - ViewManager::instance()->setRepeatMode( PlaylistModes::RepeatOne ); + AudioEngine::instance()->setRepeatMode( PlaylistModes::RepeatOne ); } break; case PlaylistModes::RepeatOne: { // switch to RepeatAll - ViewManager::instance()->setRepeatMode( PlaylistModes::RepeatAll ); + AudioEngine::instance()->setRepeatMode( PlaylistModes::RepeatAll ); } break; case PlaylistModes::RepeatAll: { // switch to NoRepeat - ViewManager::instance()->setRepeatMode( PlaylistModes::NoRepeat ); + AudioEngine::instance()->setRepeatMode( PlaylistModes::NoRepeat ); } break; @@ -611,7 +613,7 @@ AudioControls::onShuffleModeChanged( bool enabled ) void AudioControls::onShuffleClicked() { - ViewManager::instance()->setShuffled( m_shuffled ^ true ); + AudioEngine::instance()->setShuffled( m_shuffled ^ true ); } diff --git a/src/TomahawkWindow.cpp b/src/TomahawkWindow.cpp index c8c83ecb0..bf083b10e 100644 --- a/src/TomahawkWindow.cpp +++ b/src/TomahawkWindow.cpp @@ -497,6 +497,7 @@ TomahawkWindow::setupWindowsButtons() #endif } + HICON TomahawkWindow::thumbIcon(TomahawkUtils::ImageType type) { @@ -514,12 +515,6 @@ TomahawkWindow::thumbIcon(TomahawkUtils::ImageType type) void TomahawkWindow::setupSignals() { - // - connect( ViewManager::instance(), SIGNAL( repeatModeChanged( Tomahawk::PlaylistModes::RepeatMode ) ), - m_audioControls, SLOT( onRepeatModeChanged( Tomahawk::PlaylistModes::RepeatMode ) ) ); - connect( ViewManager::instance(), SIGNAL( shuffleModeChanged( bool ) ), - m_audioControls, SLOT( onShuffleModeChanged( bool ) ) ); - // connect( AudioEngine::instance(), SIGNAL( error( AudioEngine::AudioErrorCode ) ), SLOT( onAudioEngineError( AudioEngine::AudioErrorCode ) ) ); connect( AudioEngine::instance(), SIGNAL( loading( const Tomahawk::result_ptr& ) ), SLOT( onPlaybackLoading( const Tomahawk::result_ptr& ) ) );