- the playlist no more uses the mediaplayerstate as singleton

5 files changed, 35 insertions, 34 deletions

diff --git a/noncore/multimedia/opieplayer2/main.cpp b/noncore/multimedia/opieplayer2/main.cpp
index 24aba6c..02fdbf3 100644
--- a/noncore/multimedia/opieplayer2/main.cpp
+++ b/noncore/multimedia/opieplayer2/main.cpp
@@ -18,3 +18,3 @@ int main(int argc, char **argv) {
     mediaPlayerState = &st;
-    PlayListWidget pl( 0, "playList" );
+    PlayListWidget pl( st, 0, "playList" );
     playList = &pl;
diff --git a/noncore/multimedia/opieplayer2/playlistwidget.cpp b/noncore/multimedia/opieplayer2/playlistwidget.cpp
index b08204e..7ba342b 100644
--- a/noncore/multimedia/opieplayer2/playlistwidget.cpp
+++ b/noncore/multimedia/opieplayer2/playlistwidget.cpp
@@ -66,4 +66,4 @@ QString audioMimes ="audio/mpeg;audio/x-wav;audio/x-ogg";
 
-PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
-    : PlayListWidgetGui( parent, name, fl ) {
+PlayListWidget::PlayListWidget( MediaPlayerState &mediaPlayerState, QWidget* parent, const char* name )
+    : PlayListWidgetGui( mediaPlayerState, parent, name ) {
 
@@ -78,5 +78,5 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
     d->tbShuffle = new ToolButton( bar, tr( "Randomize" ),"opieplayer2/shuffle",
-                                   mediaPlayerState, SLOT( setShuffled( bool ) ), TRUE );
+                                   &mediaPlayerState, SLOT( setShuffled( bool ) ), TRUE );
     d->tbLoop    = new ToolButton( bar, tr( "Loop" ), "opieplayer2/loop",
-                                   mediaPlayerState, SLOT( setLooping( bool ) ), TRUE );
+                                   &mediaPlayerState, SLOT( setLooping( bool ) ), TRUE );
 
@@ -104,3 +104,3 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
     pmView->insertItem(  Resource::loadPixmap("fullscreen") , tr( "Full Screen"),
-                         mediaPlayerState, SLOT( toggleFullscreen() ) );
+                         &mediaPlayerState, SLOT( toggleFullscreen() ) );
 
@@ -108,3 +108,3 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
     bool b= cfg.readBoolEntry("FullScreen", 0);
-    mediaPlayerState->setFullscreen(  b );
+    mediaPlayerState.setFullscreen(  b );
     pmView->setItemChecked( -16, b );
@@ -143,7 +143,7 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
              this, SLOT( tabChanged( QWidget* ) ) );
-    connect( mediaPlayerState, SIGNAL( playingToggled( bool ) ),
+    connect( &mediaPlayerState, SIGNAL( playingToggled( bool ) ),
              d->tbPlay,  SLOT( setOn( bool ) ) );
-    connect( mediaPlayerState, SIGNAL( loopingToggled( bool ) ),
+    connect( &mediaPlayerState, SIGNAL( loopingToggled( bool ) ),
              d->tbLoop, SLOT( setOn( bool ) ) );
-    connect( mediaPlayerState, SIGNAL( shuffledToggled( bool ) ),
+    connect( &mediaPlayerState, SIGNAL( shuffledToggled( bool ) ),
              d->tbShuffle, SLOT( setOn( bool ) ) );
@@ -152,3 +152,3 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl )
     connect ( gammaSlider,  SIGNAL( valueChanged( int ) ),
-              mediaPlayerState,  SLOT( setVideoGamma( int ) ) );
+              &mediaPlayerState,  SLOT( setVideoGamma( int ) ) );
 
@@ -174,5 +174,5 @@ PlayListWidget::~PlayListWidget() {
 void PlayListWidget::initializeStates() {
-    d->tbPlay->setOn( mediaPlayerState->isPlaying() );
-    d->tbLoop->setOn( mediaPlayerState->isLooping() );
-    d->tbShuffle->setOn( mediaPlayerState->isShuffled() );
+    d->tbPlay->setOn( mediaPlayerState.isPlaying() );
+    d->tbLoop->setOn( mediaPlayerState.isLooping() );
+    d->tbShuffle->setOn( mediaPlayerState.isShuffled() );
     d->playListFrame->show();
@@ -406,4 +406,4 @@ void PlayListWidget::setDocument( const QString& fileref ) {
         d->setDocumentUsed = TRUE;
-        mediaPlayerState->setPlaying( FALSE );
-        mediaPlayerState->setPlaying( TRUE );
+        mediaPlayerState.setPlaying( FALSE );
+        mediaPlayerState.setPlaying( TRUE );
     }
@@ -431,3 +431,3 @@ const DocLnk *PlayListWidget::current() const { // this is fugly
 bool PlayListWidget::prev() {
-    if ( mediaPlayerState->isShuffled() ) {
+    if ( mediaPlayerState.isShuffled() ) {
         const DocLnk *cur = current();
@@ -445,3 +445,3 @@ bool PlayListWidget::prev() {
         if ( !d->selectedFiles->prev() ) {
-            if ( mediaPlayerState->isLooping() ) {
+            if ( mediaPlayerState.isLooping() ) {
                 return d->selectedFiles->last();
@@ -458,3 +458,3 @@ bool PlayListWidget::next() {
 //qDebug("<<<<<<<<<<<<next()");
-    if ( mediaPlayerState->isShuffled() ) {
+    if ( mediaPlayerState.isShuffled() ) {
         return prev();
@@ -462,3 +462,3 @@ bool PlayListWidget::next() {
         if ( !d->selectedFiles->next() ) {
-            if ( mediaPlayerState->isLooping() ) {
+            if ( mediaPlayerState.isLooping() ) {
                 return d->selectedFiles->first();
@@ -532,4 +532,4 @@ void PlayListWidget::playIt( QListViewItem *it) {
     if(!it) return;
-    mediaPlayerState->setPlaying(FALSE);
-    mediaPlayerState->setPlaying(TRUE);
+    mediaPlayerState.setPlaying(FALSE);
+    mediaPlayerState.setPlaying(TRUE);
     d->selectedFiles->unSelect();
@@ -616,3 +616,3 @@ void PlayListWidget::btnPlay(bool b) {
 //    mediaPlayerState->setPlaying(false);
-    mediaPlayerState->setPlaying(b);
+    mediaPlayerState.setPlaying(b);
     insanityBool=FALSE;
@@ -1051,4 +1051,4 @@ void PlayListWidget::pmViewActivated(int index) {
     {
-        mediaPlayerState->toggleFullscreen();
-        bool b=mediaPlayerState->isFullscreen();
+        mediaPlayerState.toggleFullscreen();
+        bool b=mediaPlayerState.isFullscreen();
         pmView->setItemChecked( index, b);
diff --git a/noncore/multimedia/opieplayer2/playlistwidget.h b/noncore/multimedia/opieplayer2/playlistwidget.h
index fa2837a..3f52e63 100644
--- a/noncore/multimedia/opieplayer2/playlistwidget.h
+++ b/noncore/multimedia/opieplayer2/playlistwidget.h
@@ -71,3 +71,3 @@ public:
 
-    PlayListWidget( QWidget* parent=0, const char* name=0, WFlags fl=0 );
+    PlayListWidget( MediaPlayerState &mediaPlayerState, QWidget* parent=0, const char* name=0 );
     ~PlayListWidget();
@@ -136,2 +136,3 @@ private slots:
     void playSelected();
+
 };
diff --git a/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp b/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp
index 8be7a2f..f47ddcc 100644
--- a/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp
+++ b/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp
@@ -61,6 +61,4 @@
 
-extern MediaPlayerState *mediaPlayerState;
-
-PlayListWidgetGui::PlayListWidgetGui( QWidget* parent, const char* name, WFlags fl )
-    : QMainWindow( parent, name, fl ) {
+PlayListWidgetGui::PlayListWidgetGui( MediaPlayerState &_mediaPlayerState, QWidget* parent, const char* name )
+    : QMainWindow( parent, name ), mediaPlayerState( _mediaPlayerState ) {
 
@@ -223,5 +221,5 @@ void PlayListWidgetGui::setActiveWindow()  {
     // When we get raised we need to ensure that it switches views
-    MediaPlayerState::DisplayType origDisplayType = mediaPlayerState->displayType();
-    mediaPlayerState->setDisplayType( MediaPlayerState::MediaSelection ); // invalidate
-    mediaPlayerState->setDisplayType( origDisplayType ); // now switch back
+    MediaPlayerState::DisplayType origDisplayType = mediaPlayerState.displayType();
+    mediaPlayerState.setDisplayType( MediaPlayerState::MediaSelection ); // invalidate
+    mediaPlayerState.setDisplayType( origDisplayType ); // now switch back
 }
diff --git a/noncore/multimedia/opieplayer2/playlistwidgetgui.h b/noncore/multimedia/opieplayer2/playlistwidgetgui.h
index 4eefb93..9b5252a 100644
--- a/noncore/multimedia/opieplayer2/playlistwidgetgui.h
+++ b/noncore/multimedia/opieplayer2/playlistwidgetgui.h
@@ -51,2 +51,3 @@ class PlayListWidgetPrivate;
 class PlayListSelection;
+class MediaPlayerState;
 
@@ -97,3 +98,3 @@ class PlayListWidgetGui : public QMainWindow {
 public:
-    PlayListWidgetGui( QWidget* parent=0, const char* name=0, WFlags fl=0 );
+    PlayListWidgetGui( MediaPlayerState &_mediaPlayerState, QWidget* parent=0, const char* name=0 );
     ~PlayListWidgetGui();
@@ -124,2 +125,3 @@ protected:
 
+    MediaPlayerState &mediaPlayerState;
 };