[Amarok] 2d474ee Revert "Make Amarok work correctly with Phonon-VLC

Mark Kretschmann kretschmann at kde.org
Sun Mar 28 09:42:06 UTC 2010


commit 2d474eef13f6eed9e79703f64eed3ca367478496
Author: Mark Kretschmann <kretschmann at kde.org>
Date:   Sun Mar 28 11:44:31 2010 +0200

    Revert "Make Amarok work correctly with Phonon-VLC."
    
    This reverts commit ee9792e774e2d0fd968819dc2719922ba8cf1b2b.
    
    Turns out that this commit broke Amarok with Phonon-xine, which is not
    acceptable. I'll try to find a proper solution.
    
    CCMAIL: amarok-devel at kde.org
    CCMAIL: amarok at kde.org

diff --git a/src/EngineController.cpp b/src/EngineController.cpp
index 521e570..3d57dad 100644
--- a/src/EngineController.cpp
+++ b/src/EngineController.cpp
@@ -175,10 +175,14 @@ EngineController::initializePhonon()
     connect( m_controller, SIGNAL( titleChanged( int ) ), SLOT( slotTitleChanged( int ) ) );
 
 
+    // Gapless Playback is temporarily disabled. Reason: Trying to find a bug in this code that makes
+    // the EngineController stop after each track, with all Phonon backends except the xine backend.
+#if 0
     if( AmarokConfig::trackDelayLength() > -1 )
         m_media->setTransitionTime( AmarokConfig::trackDelayLength() ); // Also Handles gapless.
     else if( AmarokConfig::crossfadeLength() > 0 )  // TODO: Handle the possible options on when to crossfade.. the values are not documented anywhere however
         m_media->setTransitionTime( -AmarokConfig::crossfadeLength() );
+#endif
 }
 
 
@@ -721,8 +725,10 @@ EngineController::setNextTrack( Meta::TrackPtr track )
         m_nextTrack = track;
         m_nextUrl = track->playableUrl();
     }
-
-    play( track );
+    else
+    {
+        play( track );
+    }
 }
 
 Phonon::State



More information about the Amarok mailing list