- initialized xine in a background thread. much time is still spend in

  skin loading though.

5 files changed, 42 insertions, 5 deletions

diff --git a/noncore/multimedia/opieplayer2/lib.cpp b/noncore/multimedia/opieplayer2/lib.cpp
index 99d5de6..20fd1e2 100644
--- a/noncore/multimedia/opieplayer2/lib.cpp
+++ b/noncore/multimedia/opieplayer2/lib.cpp
@@ -72,3 +72,5 @@ using namespace XINE;
 
-Lib::Lib( XineVideoWidget* widget ) {
+Lib::Lib( XineVideoWidget* widget ) 
+{
+    ThreadUtil::AutoLock lock( m_initGuard );
     m_initialized = false;
@@ -89,2 +91,7 @@ Lib::Lib( XineVideoWidget* widget ) {
 
+    start();
+}
+
+void Lib::run()
+{
     initialize();
@@ -126,3 +133,6 @@ void Lib::initialize()
 
+    ThreadUtil::AutoLock lock( m_initGuard );
     m_initialized = true;
+
+    send( new ThreadUtil::ChannelMessage( InitializationMessageType ), OneWay );
 }
@@ -286,3 +296,10 @@ void Lib::receiveMessage( ThreadUtil::ChannelMessage *msg, SendType sendType )
     assert( sendType == ThreadUtil::Channel::OneWay );
-    handleXineEvent( msg->type() );
+    switch ( msg->type() ) {
+        case XineMessageType:
+            handleXineEvent( static_cast<XineMessage *>( msg )->xineEvent );
+            break;
+        case InitializationMessageType:
+            emit initialized();
+            break;
+    }
     delete msg;
@@ -291,3 +308,3 @@ void Lib::receiveMessage( ThreadUtil::ChannelMessage *msg, SendType sendType )
 void Lib::handleXineEvent( const xine_event_t* t ) {
-    send( new ThreadUtil::ChannelMessage( t->type ), OneWay );
+    send( new XineMessage( t->type ), OneWay );
 }
diff --git a/noncore/multimedia/opieplayer2/lib.h b/noncore/multimedia/opieplayer2/lib.h
index 4b8dc81..34b85b9 100644
--- a/noncore/multimedia/opieplayer2/lib.h
+++ b/noncore/multimedia/opieplayer2/lib.h
@@ -57,3 +57,4 @@ namespace XINE {
     class Frame;
-    class Lib : public ThreadUtil::Channel {
+    class Lib : public ThreadUtil::Channel, private ThreadUtil::Thread
+    {
         Q_OBJECT
@@ -179,2 +180,4 @@ namespace XINE {
 
+        void initialized();
+
     protected:
@@ -182,2 +185,4 @@ namespace XINE {
 
+        virtual void run();
+
     private:
@@ -187,2 +192,13 @@ namespace XINE {
 
+        enum { XineMessageType = 1, InitializationMessageType };
+
+        struct XineMessage : public ThreadUtil::ChannelMessage
+        {
+            XineMessage( int _xineEvent ) : ThreadUtil::ChannelMessage( XineMessageType ),
+                xineEvent( _xineEvent )
+            {}
+
+            int xineEvent;
+        };
+
         mutable ThreadUtil::Mutex m_initGuard;
diff --git a/noncore/multimedia/opieplayer2/mediaplayer.cpp b/noncore/multimedia/opieplayer2/mediaplayer.cpp
index 5e91561..0d9a5b8 100644
--- a/noncore/multimedia/opieplayer2/mediaplayer.cpp
+++ b/noncore/multimedia/opieplayer2/mediaplayer.cpp
@@ -367,3 +367,4 @@ void MediaPlayer::recreateAudioAndVideoWidgets()
     xineControl = new XineControl( videoUI, videoUI->vidWidget(), mediaPlayerState );
-    mediaPlayerState.setBackendInitialized();
+    connect( xineControl, SIGNAL( initialized() ),
+             &mediaPlayerState, SLOT( setBackendInitialized() ) );
 }
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.cpp b/noncore/multimedia/opieplayer2/xinecontrol.cpp
index 0e81fa2..14e71d9 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.cpp
+++ b/noncore/multimedia/opieplayer2/xinecontrol.cpp
@@ -57,2 +57,3 @@ XineControl::XineControl( QWidget *videoContainerWidget, XineVideoWidget *xineWi
     connect( libXine, SIGNAL( stopped() ), this, SLOT( nextMedia() ) );
+    connect( libXine, SIGNAL( initialized() ), this, SIGNAL( initialized() ) );
 
diff --git a/noncore/multimedia/opieplayer2/xinecontrol.h b/noncore/multimedia/opieplayer2/xinecontrol.h
index 69a594f..000529c 100644
--- a/noncore/multimedia/opieplayer2/xinecontrol.h
+++ b/noncore/multimedia/opieplayer2/xinecontrol.h
@@ -116,2 +116,4 @@ signals:
     void positionChanged( long );
+
+    void initialized();
 };